Hello @Dvorapa.
It's hard to figure out the problem. Can you add many diffs where template has been added and diffs where it has not been ?

In T135304#3273570, @Framawiki wrote:

Hello @Dvorapa.
It's hard to figure out the problem. Can you add many diffs where template has been added and diffs where it has not been ?

I'll ask admins on cswiki for help, pages are deleted. Where CFD template was not added by pwb, I've always added it by myself.

In most cases bot doesn't add CFD template to the old page title (on the other hand removing CFR template from new page title works always), therefore I'm always surprised a bit when bot does add a CFD template to the old page title.

Perhaps you can copy the page's wikitext is a Phabricator past, before it is deleted ?

Last move:

original content on former title; with move proposal (CFR template)

{{přejmenovat kategorii|Kategorie:Inter Milán}}
{{Hlavní článek|Inter Milán}} 
* {{Commonscat}}

[[Kategorie:Italské fotbalové kluby|Internazionale]]
[[Kategorie:Sport v Miláně]]

content moved to proposed title; move proposal (CFR template) removed automatically by pwb

{{Hlavní článek|Inter Milán}}
* {{Commonscat}}

[[Kategorie:Italské fotbalové kluby|Internazionale]]
[[Kategorie:Sport v Miláně]]

redirect created automatically by pwb on the former title; CFD template NOT added

#PŘESMĚRUJ [[:Kategorie:Inter Milán]]

CFD template added by hand

{{Smazat|přejmenováno}}
#PŘESMĚRUJ [[:Kategorie:Inter Milán]]

Version: pwb core latest master for users (commit 05e0f916f280448f6ca27a7cd39f9c2652cd8aaf) obtained from gerrit and updated before move by git pull, logged in by password, python 3.6.1

Only 10/500 last category moves of my bot added CFD template like this:

original content on former title; with move proposal (CFR template)

{{Přejmenovat kategorii|Písně v jidiš}}
[[Kategorie:Písně podle jazyka]]
[[Kategorie:Jidiš]]
[[Kategorie:Židovská hudba]]

content moved to proposed title; move proposal (CFR template) removed automatically by pwb

[[Kategorie:Písně podle jazyka]]
[[Kategorie:Jidiš]]
[[Kategorie:Židovská hudba]]

redirect created automatically by pwb on the former title

#PŘESMĚRUJ [[:Kategorie:Písně v jidiš]]

CFD template added correctly, automatically by pwb

{{delete|1=Robot: nahrazení kategorie Jidiš písně za Písně v jidiš}}
#PŘESMĚRUJ [[:Kategorie:Písně v jidiš]]

But if I look into the history of deleted categories moved by bot - those 10 of last 500, where bot added CFD template, it still looks like this is a random selection.

My code always look like python pwb.py category move. I sometimes use python pwb.py category move -inplace and sometimes not, but there is no evidence of this parameter affecting the behavior

I'd like to help, my python knowledge is really good, but code for adding CFD template to the redirect is commented poorly.

Maybe category move refuses to edit the redirect always, but sometimes the check if page is redirect fails somehow? (just a guess)

Or maybe the check if category is empty fails due to server cache somehow? Because sometimes the category doesn't look empty immediately in a browser after it is emptied by bot.

Thanks for your comments @Dvorapa !

In T135304#3288800, @Dvorapa wrote:

Maybe category move refuses to edit the redirect always, but sometimes the check if page is redirect fails somehow? (just a guess)

Maybe the same problem with recognizing category redirects like in T115091?

I've got special time-limited rights on Czech Wikipedia to be able to see deleted edits of my bot (ping @Framawiki). Are there any additional information I could provide using this advantage?

if self.oldcat.isEmptyCategory() and self.delete_oldcat and \
    ((self.newcat and self.move_oldcat) or not self.newcat):
  self._delete(can_move_page, can_move_talk)

In T135304#3288814, @Dvorapa wrote:

Or maybe the check if category is empty fails due to server cache somehow? Because sometimes the category doesn't look empty immediately in a browser after it is emptied by bot.

Independently, I have made it to the same finding.

Change 355791 had a related patch set uploaded (by Matěj Suchánek; owner: Matěj Suchánek):
[pywikibot/core@master] When moving a category, track number of members locally as well

https://gerrit.wikimedia.org/r/355791

In T135304#3294544, @matej_suchanek wrote:

scripts/category.py

if self.oldcat.isEmptyCategory() and self.delete_oldcat and \
    ((self.newcat and self.move_oldcat) or not self.newcat):
  self._delete(can_move_page, can_move_talk)

I examined the same part of code when I was writing that comment, but I can not figure out a solution. Setting some time to wait is nonsense and could not solve the problem. My only thought was about some possibility to call touch on the category redirect right before this if-condition.

@matej_suchanek Your solution looks better than my thoughts, please don't forget to track exceptions like doc pages (for templates categorized through doc), skipped pages, redirects and so.

Could you please add an option to localize CFD template name too? I know {{delete}} works on most of Wikipedias, but localized one would be better maybe.

In T135304#3294954, @Dvorapa wrote:

@matej_suchanek Your solution looks better than my thoughts, please don't forget to track exceptions like doc pages (for templates categorized through doc), skipped pages, redirects and so.

In the newest patch, I improved tracking of doc subpages.

Could you please add an option to localize CFD template name too? I know {{delete}} works on most of Wikipedias, but localized one would be better maybe.

This option will add nothing since the result will be same. Localizing {{delete}} in pywikibot/page.py is considerable, though.

In T135304#3294657, @Dvorapa wrote:

My only thought was about some possibility to call touch on the category redirect right before this if-condition.

Btw there is a -rebuild parameter, maybe just calling this functionality before checking if category is empty could work as well

In T135304#3296719, @Dvorapa wrote:

In T135304#3294657, @Dvorapa wrote:

My only thought was about some possibility to call touch on the category redirect right before this if-condition.

Btw there is a -rebuild parameter, maybe just calling this functionality before checking if category is empty could work as well

Actually irrelevant.

In T135304#3295025, @matej_suchanek wrote:

In T135304#3294954, @Dvorapa wrote:

Could you please add an option to localize CFD template name too? I know {{delete}} works on most of Wikipedias, but localized one would be better maybe.

This option will add nothing since the result will be same. Localizing {{delete}} in pywikibot/page.py is considerable, though.

PS: Maybe this is already tracked in T72653

Change 355791 abandoned by Matěj Suchánek:
When moving a category, track number of members locally as well

Reason:
Unclear how to proceed

https://gerrit.wikimedia.org/r/355791