Last modified: 2014-07-24 13:11:44 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T62603, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 60603 - archivebot.py does not handle some cases of param archive= in template
archivebot.py does not handle some cases of param archive= in template
Status: RESOLVED INVALID
Product: Pywikibot
Classification: Unclassified
archivebot.py (Other open bugs)
unspecified
All All
: Unprioritized normal
: ---
Assigned To: Pywikipedia bugs
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-01-29 23:07 UTC by Mpaa
Modified: 2014-07-24 13:11 UTC (History)
2 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Mpaa 2014-01-29 23:07:55 UTC
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.
Comment 1 Yusuke Matsubara 2014-05-14 09:50:23 UTC
(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"?

2. Making the bot to follow redirection sounds like a legitimate enhancement.  Would you mind splitting it out as another request?
Comment 2 Mpaa 2014-05-14 12:37:24 UTC
(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"?
> 
> 2. 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
Comment 4 Yusuke Matsubara 2014-05-14 13:00:38 UTC
(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"?
> > 
> > 2. 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
Comment 5 Mpaa 2014-05-14 14:55:36 UTC
ooops, I did not notice that ...
Comment 6 Yusuke Matsubara 2014-07-24 12:34:01 UTC
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.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links