Last modified: 2011-02-08 21:56:24 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 T28903, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 26903 - WhatLinksHere only lists image link or transclusion, never both.
WhatLinksHere only lists image link or transclusion, never both.
Status: RESOLVED DUPLICATE of bug 26904
Product: MediaWiki
Classification: Unclassified
Special pages (Other open bugs)
1.18.x
All All
: Normal enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
http://commons.wikimedia.org/w/index....
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-01-24 16:24 UTC by Krinkle
Modified: 2011-02-08 21:56 UTC (History)
0 users

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


Attachments

Description Krinkle 2011-01-24 16:24:08 UTC
The three foreach() loops here:
http://svn.wikimedia.org/viewvc/mediawiki/trunk/phase3/includes/specials/SpecialWhatlinkshere.php?revision=80409&view=markup#l188

overwrite eachother. That was done to avoid duplicates. However the props are also rewritten.

Perhaps a check to see if it's been set already should be done and merge the is_template/is_image props.

Although we could choose to simply decide to let one count strong than the other (in this case imagelink, over templatelink, over pagelink) however the following part of the code is clearly designed with the scenario in mind that multiple props are set to 1:

http://svn.wikimedia.org/viewvc/mediawiki/trunk/phase3/includes/specials/SpecialWhatlinkshere.php?revision=80409&view=markup#l292
<pre>
298                if ( $row->is_template )
299	                        $props[] = $msgcache['istemplate'];
300	                if( $row->is_image )
301	                        $props[] = $msgcache['isimage'];
302	
303	                if ( count( $props ) ) {
304	                        $propsText = '(' . implode( $msgcache['semicolon-separator'], $props ) . ')';
305	                }
</pre>
Comment 1 Krinkle 2011-01-24 16:25:42 UTC

*** This bug has been marked as a duplicate of bug 26904 ***

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


Navigation
Links