Bugs item #1988912, was opened at 2008-06-09 06:58 Message generated for change (Comment added) made by nobody You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=603138&aid=1988912...
Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: General Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Nobody/Anonymous (nobody) Assigned to: Nobody/Anonymous (nobody) Summary: Adds revision number after custom edit-summaries
Initial Comment: Even when a custom edit-summary is provided as the second argument to
target.put(newtext,"My edit summary")
The bot will automatically add a string identifying the SVN revision to the summary
"My edit summary [r5544]"
While I can see that for some commonly used bots, some projects would like to see this information, it shouldn't be appended in the general case as it needlessly reduces the flexibility of the library.
It can be trivially fixed by changing lines 1235 and 1236 of wikipedia.py as attached but I don't have commit access.
Yours en:wikt:Conrad.Irwin
----------------------------------------------------------------------
Comment By: Nobody/Anonymous (nobody) Date: 2008-06-09 12:27
Message: Logged In: NO
@cosoleto
You can reject a bad reported bug within 1 minute, write to reporter to run version.py....
yes, /if/ the reporter is the bot owner, which not always is the case.
The system as it is now does not decrease flexibility (Yes, it adds a string. That does not disable anything...), but it does make identifying old bugs a hell of a lot easier (you *really* don't want to know how many people are using the old 'stable' revision). I would support making it a config option, or logging the revision information elsewhere.
If you would rather create a separate page 'Special:MyUser/PWB_log' or sth that logs edits/bot runs, fine with me.
----------------------------------------------------------------------
Comment By: Russell Blau (russblau) Date: 2008-06-09 12:14
Message: Logged In: YES user_id=855050 Originator: NO
I agree with removing the version number from the edit summary, except possibly for the bare default string noted by rotemliss.
----------------------------------------------------------------------
Comment By: Rotem Liss (rotemliss) Date: 2008-06-09 10:37
Message: Logged In: YES user_id=1327030 Originator: NO
I also think that the appending of the revision ID to the edit summary is not necessary.
Note that the patch suggested in this bug seems to append the revision ID only to the *default* edit summary, i.e. "Wikipedia python library". Most scripts don't use this summary, and they shouldn't.
----------------------------------------------------------------------
Comment By: Francesco Cosoleto (cosoleto) Date: 2008-06-09 10:29
Message: Logged In: YES user_id=181280 Originator: NO
I am sorry, but I don't think this commit is fine. You can reject a bad reported bug within 1 minute, write to reporter to run version.py.... A good edit summary uses a language intelligible by everybody, no messages direct to us.
I added in meta instructions about how to report a bug (http://meta.wikimedia.org/w/index.php?title=Using_the_python_wikipediabot&am...). It can be updated, translated in other languages. That is a good beginning. Please, think it over.
----------------------------------------------------------------------
Comment By: NicDumZ — Nicolas Dumazet (nicdumz) Date: 2008-06-09 09:15
Message: Logged In: YES user_id=1963242 Originator: NO
And now I see that this patch is only meant to remove the revision number on custom edit summaries, meaning that most of our scripts will still show [rXXX] after this patch, iirc. If it is the case, I'm not opposed to this patch.
Any other inputs ? valhallasw maybe ?
----------------------------------------------------------------------
Comment By: NicDumZ — Nicolas Dumazet (nicdumz) Date: 2008-06-09 09:10
Message: Logged In: YES user_id=1963242 Originator: NO
That's a behavior introduced in r5523.
While I do agree that this might be sometimes inducing, it's a very, very reliable way to know if the bot is running a decent version of the bot.
We are being reported a lot of false positives lately, and honestly, wasting an hour trying to reproduce a reported bug, when the reporting user simply comes a few days later "oh, finally I tried reading the manual : updating to the last svn version fixed my problem" is annoying.
I personally think, as both a developer and a bot owner, that the induced bot owner disagreements are minor compared to the time the developers will save. I might be wrong on that point, and any example of a very annoying edit summary case is welcome to prove me wrong; but until then, I don't think that this change is going to be reverted.
Thanks...
----------------------------------------------------------------------
You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=603138&aid=1988912...