Bugs item #2564757, was opened at 2009-02-04 18:47 Message generated for change (Settings changed) made by xqt 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: Closed Resolution: Fixed Priority: 5 Private: No Submitted By: Nobody/Anonymous (nobody) Assigned to: xqt (xqt)
Summary: 2219082
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: xqt (xqt) Date: 2009-09-15 18:39
Message: r7237
----------------------------------------------------------------------
Comment By: Nobody/Anonymous (nobody) Date: 2009-02-24 14:18
Message: I think this is solved in patch request 2219082.
[w:de:User:Xqt]
----------------------------------------------------------------------
Comment By: Nobody/Anonymous (nobody) Date: 2009-02-24 14: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...