Last modified: 2013-09-25 09:00:40 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 T56416, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 54416 - http://sourceforge.net/p/pywikipediabot/feature-requests/349/ is not a valid "see also" url
http://sourceforge.net/p/pywikipediabot/feature-requests/349/ is not a valid ...
Status: RESOLVED FIXED
Product: Wikimedia
Classification: Unclassified
Bugzilla (Other open bugs)
wmf-deployment
All All
: High normal (vote)
: ---
Assigned To: Kunal Mehta (Legoktm)
http://sourceforge.net/p/pywikipediab...
: upstream
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-09-21 03:39 UTC by Kunal Mehta (Legoktm)
Modified: 2013-09-25 09:00 UTC (History)
7 users (show)

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


Attachments

Description Kunal Mehta (Legoktm) 2013-09-21 03:39:02 UTC
Go to https://bugzilla.wikimedia.org/show_bug.cgi?id=54415

Try and add "http://sourceforge.net/p/pywikipediabot/feature-requests/349/" as a see also link

Get "http://sourceforge.net/p/pywikipediabot/feature-requests/349/ is not a valid URL to a bug. See Also URLs should point to one of: <snip" back in a nice red warning.

All of our urls will be formed like that, but the "feature-requests" part can also be: 'patches', 'support-requests', or 'bugs'.
Comment 1 Kunal Mehta (Legoktm) 2013-09-21 03:48:58 UTC
Probably an upstream bug since sf recently changed their url structure:

https://bugzilla.mozilla.org/show_bug.cgi?id=624522
Comment 2 Gerrit Notification Bot 2013-09-21 03:53:27 UTC
Change 85399 had a related patch set uploaded by Legoktm:
Add support for sf.net Allura URLs to Bugzilla's 'See also' field

https://gerrit.wikimedia.org/r/85399
Comment 3 Kunal Mehta (Legoktm) 2013-09-21 04:30:09 UTC
Filed upstream at https://bugzilla.mozilla.org/show_bug.cgi?id=919122 and attempted to submit a patch.
Comment 4 Gerrit Notification Bot 2013-09-24 13:42:41 UTC
Change 85399 merged by Dzahn:
Add support for sf.net Allura URLs to Bugzilla's 'See also' field

https://gerrit.wikimedia.org/r/85399
Comment 5 Daniel Zahn 2013-09-24 13:52:31 UTC
seems it doesn't match properly yet. Because we now list sourceforge bugs as valid but i couldn't add a random one like:

http://sourceforge.net/p/pywikipediabot/bugs/1674/ is not a valid URL to a bug.
Comment 7 Andre Klapper 2013-09-24 15:17:23 UTC
I'll leave this ticket open and it's up to Legoktm whether to write a followup patch to attach an optional / at the end of the URL. :)

Thanks Daniel for merging Legoktm's patch (so work to import pywikipedia reports from SourceForge to Bugzilla can continue)!
Comment 8 Amir Ladsgroup 2013-09-24 15:23:51 UTC
Or we can take care of this in the migrating script:
https://gerrit.wikimedia.org/r/#/q/project:pywikibot/sf-export,n,z

Thanks
Best
Comment 9 Gerrit Notification Bot 2013-09-24 17:23:12 UTC
Change 85883 had a related patch set uploaded by Legoktm:
Optional / at the end of the url

https://gerrit.wikimedia.org/r/85883
Comment 10 Kunal Mehta (Legoktm) 2013-09-24 17:26:30 UTC
I also modified the import script in https://gerrit.wikimedia.org/r/#/c/85884/ to strip the trailing /, so this is no longer a blocker.

I'll run the import in a few hours once I'm on a more stable internet connection.
Comment 11 Kunal Mehta (Legoktm) 2013-09-24 21:29:17 UTC
Bleh, https://sourceforge.net/p/pywikipediabot/feature-requests/349 does not work either since the '-' in feature-requests isn't matched in the regex. I'll amend the current patch to cover that.
Comment 12 Gerrit Notification Bot 2013-09-25 08:57:33 UTC
Change 85883 merged by Dzahn:
Optional / at the end of the url, allow for - to be in bugtype

https://gerrit.wikimedia.org/r/85883
Comment 13 Daniel Zahn 2013-09-25 09:00:40 UTC
merged. And now i could add the URL from the bug title as "See also". resolved

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


Navigation
Links