Last modified: 2008-11-20 05:45:48 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 T18299, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 16299 - Recursion depth in SMWResultPrinter as configuration parameter
Recursion depth in SMWResultPrinter as configuration parameter
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
Semantic MediaWiki (Other open bugs)
unspecified
All All
: Normal enhancement (vote)
: ---
Assigned To: Markus Krötzsch
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-11-10 17:36 UTC by Maxim Ilinykh
Modified: 2008-11-20 05:45 UTC (History)
0 users

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


Attachments

Description Maxim Ilinykh 2008-11-10 17:36:57 UTC
It would be very useful to have maximum recursion depth (see $mRecursionDepth field of SMWResultPrinter class being used in getResult method) being defined in SMW_Settings.php as a separate configuration parameter.

I'm using four nested template calls to display complex query result. It was possible in SMW 1.1.2 but it's not possible now.
Comment 1 Markus Krötzsch 2008-11-19 17:39:51 UTC
Ok, wish granted. You can set, e.g., 

 SMWResultPrinter::$maxRecursionDepth = 5;

in LocalSettings.php *after* enableSemantics() was called. Note that recursive templates can easily be crafted to increase the results size exponentially, so use this setting with care. If some recursion goes too far, then you will get a PHP memory allocation error or simply a blank result page -- directly access the URL with action=edit in this case to fix its contents.
Comment 2 Maxim Ilinykh 2008-11-20 05:45:48 UTC
Thanks! It's exactly what I wanted!

And thanks for warning - I'll be careful with recursive templates.

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


Navigation
Links