Patches item #3147585, was opened at 2010-12-29 05:55 Message generated for change (Comment added) made by valhallasw You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=603140&aid=3147585...

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: 6 Private: No Submitted By: DixonD (dixond) Assigned to: xqt (xqt) Summary: Working with interwikis on subpage

Initial Comment: I think I implemented my feature request with ID: 3146291 Please, review carefully my code - to be honest I don't have really much experience with Python. And few moments: 1) When we adding/fixing interwikis on subpage, I think we should remove interwikis in template itself if any 2) It seems that logic of determing whether this page has subpages, getting interwikis also from included subpages etc should be decoupled from interwiki.py and moved to wikipedia.py 3) Not really related but.. cosmetic_changes.py removes interwikis from subpage.

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

Comment By: Merlijn S. van Deen (valhallasw)

Date: 2012-03-21 10:00

Message: Sorry for the very... slow response. The problem is that interwiki.py is complicated (as you have noticed), and it's a somewhat complicated patch. To be honest: I'm not even sure how much interwiki.py does in terms of templates, subpages... and as such I'm scared as hell to even touch the script, considering it sort of works at the moment.

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

Comment By: DixonD (dixond) Date: 2011-03-30 02:37

Message: Any news?

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

Comment By: DixonD (dixond) Date: 2010-12-30 04:46

Message: Uploaded new patch with some fixes.

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

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