Last modified: 2013-04-22 16:17:21 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 T44658, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 42658 - Display units not respected
Display units not respected
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
Semantic MediaWiki (Other open bugs)
master
All All
: Unprioritized normal (vote)
: ---
Assigned To: MWJames
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-12-03 20:14 UTC by Leonard Wallentin
Modified: 2013-04-22 16:17 UTC (History)
6 users (show)

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


Attachments

Description Leonard Wallentin 2012-12-03 20:14:02 UTC
At some point display units stopped working for me, in the sense that ALL units of a quantity property is printed out. I have not been able to reproduce this on referata, but looking for the cause, I found an odd thing at line 180 in inincludes/datavalues/SMW_DV_Quantity.php

It currently reads:
 
 if ( $di->getDIType() instanceof SMWDIBlob ) {

...but $di->getDIType will be an integer (3, if SMWDIBlob). I think it is supposed to be like this:

 if ( $di instanceof SMWDIBlob ) {

Changing it like that makes display units work again, at least for me!
Comment 1 MWJames 2012-12-03 20:32:33 UTC
Yes, you are right and the winner for the first bug found after release. Will have a fix ready after the 1.9.x and 1.8.x branch have been swapped.
Comment 2 Leonard Wallentin 2012-12-03 20:52:50 UTC
Yay! :)
Comment 3 MWJames 2012-12-06 03:07:30 UTC
https://gerrit.wikimedia.org/r/#/c/37182/1
Comment 4 MWJames 2012-12-13 12:57:24 UTC
Merged into 1.9.x and 1.8.x
Comment 5 Quim Gil 2012-12-13 15:07:29 UTC
Can you please assign the bugs you mark as ASSIGNED or FIXED? Clicking the "take" link adds just an extra second to your work, improves our workflow and avoids any chance to get pointless notifications like this one.  :)

Background & discussion: http://lists.wikimedia.org/pipermail/wikitech-l/2012-December/064874.html - please follow-up there is you wish. Thank you!

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


Navigation
Links