Closed (fixed)
Project:
Mollom
Version:
6.x-1.15
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
28 Mar 2011 at 09:53 UTC
Updated:
24 Apr 2014 at 17:13 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
kenorb commentedComment #2
kenorb commentedThe same with Filefield: #1105064: FileField: Test: Fatal error: Maximum function nesting level of '400' reached, aborting! in filefield.test with PHP 5.3
Comment #3
kenorb commentedSo assuming that's general problem of SimpleTest.
Comment #4
rfayI'd be surprised. Do you *want* 400 levels of recursion in your test? I'd recommend taking a look at your test.
I'm assuming Simpletest is complaining about recursion. And your included result looks like recursion.
Comment #5
rfayPlease make sure you use the latest dev of simpletest for D6, as a number of commits have happened since the last stable release.
Comment #6
sunI'm not sure what is going on here, but it clearly seems that self::setUp() is repetitively called instead of parent::setUp():
The inheritance looks this way:
This code hasn't been changed for a long time and there isn't really anything special about it.
Since @kenorb reported the same error also for FileField tests, it seems obvious that this is not a problem with this module's tests.
Comment #7
rfayOK, well I'll put it back then. Hmm. Pretty weird. And I assume we're using the dev of simpletest...
Comment #8
kenorb commentedThis happens only with PHP 5.3.x (Tested with 5.3.5).
Comment #9
kenorb commentedFollowing solution which I found work:
Source: http://php.net/manual/en/function.call-user-func-array.php
Comment #10
kenorb commentedThanks for your help.
Attached patch fix the problem for Mollom 1.15 (latest stable release).
I'm not sure if it's some kind of PHP bug, or improper syntax.
Comment #11
kenorb commentedI see that in dev, the code is different now. It was fixed in another way.
http://drupalcode.org/project/mollom.git/blobdiff/e44ea30d05bd1f33ccb95f...
Comment #12
kenorb commentedComment #13
kenorb commentedReported similar predictable bug for Admin Menu: #1146644: Admin Menu: Test: PHP 5.3 compability for parent::setUp callback
Following modules are affected by that syntax:
Last two (project project_issue) and are not affected anymore.
Comment #14
kenorb commentedProject and project_issue module past bug reports:
#877562: PHP 5.3 compatibility for tests
#992892: Tests not working with PHP 5.3
Comment #15
kenorb commentedCreated two bug reports:
#1146670: ImageField: PHP 5.3 compatibility for tests
#1146672: Mentions: PHP 5.3 compatibility for tests
Comment #16
kenorb commentedComment #17
rfayThanks for the great work on this.