Last modified: 2009-07-23 22:56:17 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 T21817, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 19817 - CooperationStatistics - includable charts
CooperationStatistics - includable charts
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
CooperationStatistics (Other open bugs)
unspecified
All All
: Normal enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
http://www.mediawiki.org/wiki/Extensi...
: patch
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-07-19 03:52 UTC by Al Maghi
Modified: 2009-07-23 22:56 UTC (History)
1 user (show)

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


Attachments
includable charts (1.95 KB, patch)
2009-07-19 03:52 UTC, Al Maghi
Details
includable charts - alternate (915 bytes, patch)
2009-07-19 03:56 UTC, Al Maghi
Details
includable charts /correct (1.67 KB, patch)
2009-07-19 04:03 UTC, Al Maghi
Details

Description Al Maghi 2009-07-19 03:52:44 UTC
Created attachment 6362 [details]
includable charts
Comment 1 Al Maghi 2009-07-19 03:56:12 UTC
Created attachment 6363 [details]
includable charts - alternate

+ if( !$this->mIncluding )

in
  function OutputTableRaw() 
rather than in
  function execute()

I don't know which one is better.
Comment 2 Al Maghi 2009-07-19 04:03:51 UTC
Created attachment 6364 [details]
includable charts    /correct
Comment 3 Al Maghi 2009-07-21 23:15:21 UTC
in 6363 mIncluding test is in OutputTableRaw()

in 6364 mIncluding test is in execute()
Comment 4 Alexandre Emsenhuber [IAlex] 2009-07-23 21:06:48 UTC
applied fixes fixes in r53699.
Comment 5 Alexandre Emsenhuber [IAlex] 2009-07-23 21:07:23 UTC
sorry it's "with fixes" :)

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


Navigation
Links