Last modified: 2013-05-14 11:01:35 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 T49123, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 47123 - SMWAggregatablePrinter fails to aggregate numbers
SMWAggregatablePrinter fails to aggregate numbers
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
Semantic MediaWiki (Other open bugs)
unspecified
All All
: Unprioritized normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-04-11 13:07 UTC by MWJames
Modified: 2013-05-14 11:01 UTC (History)
4 users (show)

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


Attachments

Description MWJames 2013-04-11 13:07:47 UTC
Based on an email [1] from  Krabina Bernhard and after some investigation SMWAggregatablePrinter was suppose to generate aggregate numbers for the following sample data set. 

## Example

Subject = Test

[[AA::4]]
[[BB::2]]
[[CC::16]]
[[DD::0]]
[[EE::5 ]]
[[FF::3]]
[[GG::2]]

[[Category:Test123]]

{{#ask: [[Category:Test123]]
|?AA
|?BB
|?CC
|?DD
|?EE
|?FF
|?GG
|format=...
}}

SMWAggregatablePrinter::getNumericResults would output (for subject "Test")

array(1) {
  ["Test"]=>
  float(2) ---> wrong number
}

and for mainlabel=-

array(1) {
  [4]=>   ---> no label
  float(2) ---> wrong number
}

Where in fact expected results for subject "Test" are

array(1) {
  ["Test"]=>  ---> expected label
  float(32)   ---> expected result (aggregate numbers for Test)
}

and for mainlabel=-

array(6) {
  ["BB"]=>
  float(2)
  ["CC"]=>
  float(16)
  ["DD"]=>
  float(0)
  ["EE"]=>
  float(5)
  ["FF"]=>
  float(3)
  ["GG"]=>
  float(2)
}

[1] http://sourceforge.net/mailarchive/message.php?msg_id=30710813
Comment 1 Gerrit Notification Bot 2013-04-11 13:09:15 UTC
Related URL: https://gerrit.wikimedia.org/r/58689 (Gerrit Change I93296527981efd8c31cb7e2ba95ca688f00ddb4a)
Comment 2 Gerrit Notification Bot 2013-04-11 13:09:17 UTC
Related URL: https://gerrit.wikimedia.org/r/58689 (Gerrit Change I93296527981efd8c31cb7e2ba95ca688f00ddb4a)
Comment 3 Gerrit Notification Bot 2013-04-19 15:06:38 UTC
https://gerrit.wikimedia.org/r/58689 (Gerrit Change I93296527981efd8c31cb7e2ba95ca688f00ddb4a) | change APPROVED and MERGED [by jenkins-bot]
Comment 4 Andre Klapper 2013-05-14 09:24:56 UTC
Gerrit patch got merged, can this bug report get closed as RESOLVED FIXED or is there anything else to do?

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


Navigation
Links