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.
When I tried to delete a comment yesterday, the resulting error was
Fatal error: Call to undefined function: ctype_digit() in /var/www/drupal.org/htdocs/modules/comment.module on line 881
Related to the recent modification to comment.module.
Comment | File | Size | Author |
---|---|---|---|
#7 | comment.module_r1.455.2_0.5_patch01 | 2.16 KB | AjK |
#4 | comment.module_r1.455.2.5_patch01 | 2.16 KB | AjK |
Comments
Comment #1
AjK CreditAttribution: AjK commentedThe requirements page (http://drupal.org/requirements) says Drupal 4.6 and beyond need PHP 4.3.3+
The ctype_digit() functions are built-in by default from PHP 4.2+ so if it's unavailable to you, I guess you running a version of PHP older than 4.2 ?
I can roll a patch back to is_numeric() if people are going to be having problems like this?
What's the concensus on this?
best regards,
-AjK
Comment #2
Heine CreditAttribution: Heine commentedThis is about drupal.org :)
Comment #3
Dries CreditAttribution: Dries commentedI wonder how that patch got committed. I asked not to use ctype_digit ... we'll have to fix core.
Comment #4
AjK CreditAttribution: AjK commentedchange ctype_digit() to is_numeric() for DRUPAL-4-7
best regards,
--AjK
Comment #5
drummctype_digit() seemed like it was the right function, but enough distributions don't have it enabled by default for this to be a problem.
I think something line-wrapped the patch and it isn't working for me.
Comment #6
drummPathauto uses a ctype function and is enabled on groups.drupal.org.
I've had to recompile PHP on Drupal.org previously for this (the main issue was with distributed logins not working due to XML not being compiled in) with chx's advice. At the time no OSL people could be found.
Comment #7
AjK CreditAttribution: AjK commentedThat's odd. The file isn't line wrapped on my unix machine but when you open it on Windows (via samba) it gets wrapped (must make a note of that)
Let's try again....
best regards,
--AjK
Comment #8
drummCommitted to HEAD.
Since this is an infrastructure issue, I'll leave this issue open.
There is the same problem with pathauto on groups.
Comment #9
killes@www.drop.org CreditAttribution: killes@www.drop.org commentedcommitted to 4.7 and updated drupal.org
Comment #10
(not verified) CreditAttribution: commented