Last modified: 2012-05-03 21:54:08 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 T35536, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 33536 - Setting to disable PediaPress integration?
Setting to disable PediaPress integration?
Status: VERIFIED FIXED
Product: MediaWiki extensions
Classification: Unclassified
Collection (Other open bugs)
unspecified
All All
: Unprioritized enhancement (vote)
: ---
Assigned To: Marcin Cieślak
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-01-05 10:41 UTC by varnent
Modified: 2012-05-03 21:54 UTC (History)
6 users (show)

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


Attachments
Proposed patch for hiding "Print with PediaPress" (215 bytes, patch)
2012-03-13 11:45 UTC, Cristian Consonni
Details

Description varnent 2012-01-05 10:41:37 UTC
While I think a vast majority of wikis using the Collections extension want PediaPress integration, it has been pointed out (https://www.mediawiki.org/wiki/Thread:Project:Support_desk/How_do_I_remove_the_option_for_Pedia_Press_to_print_books_in_Collection%3F_IMPORTANT) that its integration presents a legal problem for some wikis that would like to utilize the other features of the Collections extension.
Any chance of adding a setting to disable PediaPress integration?
Comment 1 Cristian Consonni 2012-03-13 11:45:12 UTC
Created attachment 10227 [details]
Proposed patch for hiding "Print with PediaPress"

While I think that the best solution would be a configuration variable, I have attached a patch which works around this issue hiding the box "Print with PediaPress" (i.e. make it not appear in Special:Book).
Comment 2 p858snake 2012-03-13 11:56:43 UTC
+patch+need-review

Mini-review: We should really do this properly with a configuration variable compared to a css hiding due to the concerns linked to c0.
Comment 3 Ralf Schmitt 2012-03-13 12:27:48 UTC
my children need something to eat! we can't turn that off.

seriously, I'd like a way to configure the mPODPartners and turn the functionality off, if mPODPartners is empty. 

side note: please provide patches in unified diff format.
Comment 4 Marcin Cieślak 2012-03-13 13:34:36 UTC
Working on it. The code isn't really  ready for multiple $this->mPODPartners. Do you mind if I commit my change to SVN trunk for review when done?
Comment 5 Ralf Schmitt 2012-03-13 13:57:24 UTC
Isn't that a bit late for *review*?

Bit I'm no code review nazi, so feel free to go ahead.
Comment 6 Marcin Cieślak 2012-03-13 18:26:38 UTC
Fixed in r113740.

Now you can do:

$wgCollectionPODPartners["mojapress"] = array(
        'name' => 'MojaPress',
        'url' => 'http://mojapress.com/',
        'posturl' => 'http://mojapress.com/api/collections/',
);
$wgCollectionPODPartners["trzeciapress"] = array(
        'name' => 'TrzeciaPress',
        'url' => 'http://trzeciapress.com/',
        'posturl' => 'http://trzeciapress.com/api/collections/',
        'infopagetitle' => 'coll-trzeciapress_article'
);

or, if you really don't want it:

$wgCollectionPODPartners = false;

in your LocalSettings.php
Comment 7 Ralf Schmitt 2012-03-13 19:40:57 UTC
Thanks Marcin.

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


Navigation
Links