Last modified: 2014-01-05 16:18:44 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 T52808, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 50808 - There is no need for a "details" link if the user is already in the "details" page
There is no need for a "details" link if the user is already in the "details"...
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
AbuseFilter (Other open bugs)
unspecified
All All
: Low minor (vote)
: ---
Assigned To: sakshi.april5
https://en.wikipedia.org/wiki/Special...
: easy, patch, patch-need-review
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-07-05 15:21 UTC by Helder
Modified: 2014-01-05 16:18 UTC (History)
5 users (show)

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


Attachments
I have commented the line so that the actionLinks does not have the detailsLink. (583 bytes, patch)
2013-10-29 12:50 UTC, sakshi.april5
Details

Description Helder 2013-07-05 15:21:19 UTC
When a user visits a page such as [[Special:AbuseLog/9999]] the extension adds two links:
------------------------------------------------------------
18:01, 19 March 2009: 119.94.161.169 (talk) triggered filter 30, performing the action "edit" on Database. Actions taken: Warn; Filter description: Large deletion from article by new editors (details | examine)
------------------------------------------------------------
The link whose text is "details" points to that same page, so it is just cluttering the interface.
Comment 1 sakshi.april5 2013-10-29 10:30:02 UTC
Hello,

I am a beginner and would like to work on this bug. Can somebody assign it to me?
Comment 3 sakshi.april5 2013-10-29 12:50:28 UTC
Created attachment 13612 [details]
I have commented the line so that the actionLinks does not have the detailsLink.
Comment 4 Andre Klapper 2013-10-29 14:03:51 UTC
Hi! Thanks for your patch!

Please use Developer access
  https://www.mediawiki.org/wiki/Developer_access
to submit this as a Git branch directly into Gerrit:
  https://www.mediawiki.org/wiki/Git/Tutorial

Putting your branch in Git makes it easier to review it quickly.
Thanks again! We appreciate your contribution.
Comment 5 Nischay Nahata 2013-10-30 05:26:19 UTC
Why comment and not remove the line entirely if its not required?
Comment 6 sakshi.april5 2013-10-30 10:33:43 UTC
Well while trying to fix the bug, I was not sure if this is the correst patch or not. Hence I decided to just commment the line and not remove it completely
Comment 7 Nischay Nahata 2013-10-30 11:54:39 UTC
Ok. As Andre said please try using Git and Gerrit to submit a patch. It will be reviewed there so you can remove it if needed.
Comment 8 sakshi.april5 2013-10-30 13:47:21 UTC
Well I tried to work with Git and Gerrit to submit the patch. 
But I am unable to submit a patch. 

$git review -R

ssh: connect to host gerrit.wikimedia.org port 29418: Connection timed out
fatal: The remote end hung up unexpectedly

I always get this error so unable to submit the patch
Comment 9 Andre Klapper 2013-10-30 14:15:53 UTC
Might be bug 56034 comment 8 - might work after a few times of re-trying.
Comment 10 sakshi.april5 2013-10-30 14:30:53 UTC
Well I tried it a few times. 
In the main directory there exists a .git/ directory and .gitreview file, but still git review -R does not work.
Comment 11 Gerrit Notification Bot 2014-01-03 11:12:42 UTC
Change 105164 had a related patch set uploaded by 01tonythomas:
Removed the "details" link if the user is already in details page

https://gerrit.wikimedia.org/r/105164
Comment 12 Gerrit Notification Bot 2014-01-05 14:42:28 UTC
Change 105164 merged by jenkins-bot:
Removed the "details" link if the user is already in details page

https://gerrit.wikimedia.org/r/105164

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


Navigation
Links