Bugs item #2564757, was opened at 2009-02-04 17:47 Message generated for change (Comment added) made by nobody You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=603138&aid=2564757...

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: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Nobody/Anonymous (nobody) Assigned to: Nobody/Anonymous (nobody) Summary: redirect.py edits twice

Initial Comment: If redirect-bot detects a self loop in method fix_double_redirects, it edits twice if sd_template and sd_tagging_sum is defined for the processed language. One for putting the template and one fixing the redirect which is wrong. Of cause it removes the linefeed behind the template the second edit will be done. This diff-link gives an example for that: [http://en.wikipedia.org/w/index.php?title=Talk:Vitamin_K_reactions&diff=...] A quick and dirty fix would be changing the line

else: break # TODO Better implement loop redirect

to

break # TODO Better implement loop redirect

(delete else: and reduce 1 indention)

but there are no tests or exception handling. I think this is meant by "# TODO Better implement loop redirect"

[w:de:User:Xqt]

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody) Date: 2009-02-24 13:18

Message: I think this is solved in patch request 2219082.

[w:de:User:Xqt]

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody) Date: 2009-02-24 13:17

Message: I think this is solved in patch request 2219082.

[w:de:User:Xqt]

----------------------------------------------------------------------

You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=603138&aid=2564757...