Page MenuHomePhabricator

archivebot.py does not handle some cases of param archive= in template
Closed, InvalidPublic

Description

Cases like this are not supported, see https://en.wikipedia.org/wiki/Talk:15th_%28Imperial_Service%29_Cavalry_Brigade

{{User:MiszaBot/config

algo=old(7d)
archive={{subst:15th (Imperial Service) Cavalry Brigade}}/Archive %(counter)d
counter=1
maxarchivesize=70K
archiveheader={{talk archive navigation}}
minthreadstoarchive=1
minthreadsleft=0

}}

Another case is when page has been moved and archive= is the old name (i.e. a redirect to the new name.


Version: unspecified
Severity: normal

Details

Reference
bz60603

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 2:55 AM
bzimport set Reference to bz60603.
bzimport added a subscriber: Unknown Object (????).

(In reply to Mpaa from comment #0)

Cases like this are not supported, see
https://en.wikipedia.org/wiki/Talk:
15th_%28Imperial_Service%29_Cavalry_Brigade

{{User:MiszaBot/config

algo=old(7d)
archive={{subst:15th (Imperial Service) Cavalry Brigade}}/Archive

%(counter)d

counter=1
maxarchivesize=70K
archiveheader={{talk archive navigation}}
minthreadstoarchive=1
minthreadsleft=0

}}

Another case is when page has been moved and archive= is the old name (i.e.
a redirect to the new name.

  1. If the request is to allow templates to be used inside the "archive" argument, I'm not sure if there are many use cases. The provided example doesn't seem to be a good one to explain the need. I don't see what you expected by having "archive={{subst:15th (Imperial Service) Cavalry Brigade}}/Archive %(counter)d". Why not simply use "archive=Talk:15th (Imperial Service) Cavalry Brigade}}/Archive %(counter)d"?
  1. Making the bot to follow redirection sounds like a legitimate enhancement. Would you mind splitting it out as another request?

(In reply to Yusuke Matsubara from comment #1)

  1. If the request is to allow templates to be used inside the "archive"

argument, I'm not sure if there are many use cases. The provided example
doesn't seem to be a good one to explain the need. I don't see what you
expected by having "archive={{subst:15th (Imperial Service) Cavalry
Brigade}}/Archive %(counter)d". Why not simply use "archive=Talk:15th
(Imperial Service) Cavalry Brigade}}/Archive %(counter)d"?

  1. Making the bot to follow redirection sounds like a legitimate

enhancement. Would you mind splitting it out as another request?

On Point 1
I submitted this after trying to do what you did on that Talk page. My edit was reverted, see https://en.wikipedia.org/w/index.php?title=Talk:15th_%28Imperial_Service%29_Cavalry_Brigade&action=history

(In reply to Mpaa from comment #2)

(In reply to Yusuke Matsubara from comment #1)

  1. If the request is to allow templates to be used inside the "archive"

argument, I'm not sure if there are many use cases. The provided example
doesn't seem to be a good one to explain the need. I don't see what you
expected by having "archive={{subst:15th (Imperial Service) Cavalry
Brigade}}/Archive %(counter)d". Why not simply use "archive=Talk:15th
(Imperial Service) Cavalry Brigade}}/Archive %(counter)d"?

  1. Making the bot to follow redirection sounds like a legitimate

enhancement. Would you mind splitting it out as another request?

On Point 1
I submitted this after trying to do what you did on that Talk page. My edit
was reverted, see
https://en.wikipedia.org/w/index.php?title=Talk:
15th_%28Imperial_Service%29_Cavalry_Brigade&action=history

Ah, I see. This fix [1] should work. (your edit didn't remove the parentheses, and my Comment #1, too.)

[1] https://en.wikipedia.org/w/index.php?title=Talk:15th_%28Imperial_Service%29_Cavalry_Brigade&diff=608539436&oldid=593019435

ooops, I did not notice that ...

So It was not a problem on the bot's side.

(In reply to Mpaa from comment #0)

Another case is when page has been moved and archive= is the old name (i.e.
a redirect to the new name.

This point was not related thus not addressed. I invite another request if needed.