Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Following error appears when run simpletest for Mollom:
Fatal error: Maximum function nesting level of '400' reached, aborting! in tests/mollom.test on line 83
Comment | File | Size | Author |
---|---|---|---|
#10 | mollom.test.patch | 544 bytes | kenorb |
Comments
Comment #1
kenorb CreditAttribution: kenorb commentedComment #2
kenorb CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: kenorb commentedThis happens only with PHP 5.3.x (Tested with 5.3.5).
Comment #9
kenorb CreditAttribution: kenorb commentedFollowing solution which I found work:
Source: http://php.net/manual/en/function.call-user-func-array.php
Comment #10
kenorb CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: kenorb commentedComment #13
kenorb CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: kenorb commentedCreated two bug reports:
#1146670: ImageField: PHP 5.3 compatibility for tests
#1146672: Mentions: PHP 5.3 compatibility for tests
Comment #16
kenorb CreditAttribution: kenorb commentedComment #17
rfayThanks for the great work on this.