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.
You may want to note on the project page and readme that there is a PHP 5.3.x requirement for the module or add in some workarounds. I ran into an lcfirst() issue testing http://drupal.org/project/qa_checklist which tipped me off.
Comments
Comment #1
TravisCarden CreditAttribution: TravisCarden commentedThanks for the report, @chales! I'd rather make the module compatible with PHP 5.2.
lcfirst()
is an easy one to fix. I'll see if there are any others and then commit a fix. I appreciate the feedback!Comment #2
chales CreditAttribution: chales commentedI did change
to
but I received another error immediately so I assumed there were more incompatibilities and switched to testing under a 5.3. I'll take another look and roll a patch for you.
Comment #3
TravisCarden CreditAttribution: TravisCarden commentedA patch would be much appreciated while I try to get a PHP 5.2 environment to test against. Thanks!
Comment #4
TravisCarden CreditAttribution: TravisCarden commentedI got a test environment, and I didn't find any issues other than
lcfirst()
. The change has been committed. Thanks for the help!Comment #5
chales CreditAttribution: chales commentedConfirmed! It looks like the other issues are in qa_checklist.
Thanks for the project. I've already come up with a number of ways to put it to good use.
Comment #6
TravisCarden CreditAttribution: TravisCarden commentedThank you, @chales. I haven't found any PHP 5.3 issues in qa_checklist. If you have, please file an issue for them in that queue.
I'm glad you're putting the project to use! Please keep giving me feedback so I can make it better! :)