Last modified: 2006-11-04 18:01:33 UTC

Wikimedia Bugzilla is closed!

Wikimedia has migrated from Bugzilla to Phabricator. Bug reports should be created and updated in Wikimedia Phabricator instead. Please create an account in Phabricator and add your Bugzilla email address to it.
Wikimedia Bugzilla is read-only. If you try to edit or create any bug report in Bugzilla you will be shown an intentional error message.
In order to access the Phabricator task corresponding to a Bugzilla report, just remove "static-" from its URL.
You could still run searches in Bugzilla or access your list of votes but bug reports will obviously not be up-to-date in Bugzilla.
Bug 7801 - add regression test support for parser functions
add regression test support for parser functions
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
General/Unknown (Other open bugs)
1.9.x
PC Linux
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
: patch, patch-need-review
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2006-11-03 15:32 UTC by Steve Sanbeg
Modified: 2006-11-04 18:01 UTC (History)
0 users

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


Attachments
patch to support parser functions (1.74 KB, patch)
2006-11-03 15:33 UTC, Steve Sanbeg
Details
update to supress warning (1.84 KB, patch)
2006-11-03 18:10 UTC, Steve Sanbeg
Details

Description Steve Sanbeg 2006-11-03 15:32:00 UTC
The regression test suite is really useful, and only needs a minor enhancement
to support parser functions.
Comment 1 Steve Sanbeg 2006-11-03 15:33:33 UTC
Created attachment 2627 [details]
patch to support parser functions

This patch allows loading parser functions, i.e.

!! functionhooks
my-tag
!! endfunctionhooks
Comment 2 Brion Vibber 2006-11-03 15:51:33 UTC
This produces a PHP warning when run and there's no !!functionhooks section:

Warning: Invalid argument supplied for foreach() in /opt/web/pages/trunk/maintenance/parserTests.inc 
on line 281

Make sure you test with error_reporting set to E_ALL.
Comment 3 Steve Sanbeg 2006-11-03 18:10:25 UTC
Created attachment 2628 [details]
update to supress warning

OK, it's fixed, and tested with the default file and one that included the
parser functions.
Comment 4 Brion Vibber 2006-11-04 18:01:33 UTC
Thanks! Applied on trunk in r17410

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


Navigation
Links