Just enabled pathauto which I use with a minimalist PHP install and found out that it won't function without ctype enabled.

We have --disable-ctype in our PHP configuration script run.

Could you please use some of the PHP core functions instead if possible? I don't believe that Drupal itself has the ctype requirements.

Comments

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Version: 5.x-1.1 » 5.x-2.x-dev

Perhaps you could provide a patch to do this? If so I will be happy to commit it.

It works fine for so many people, so for my own time this is a lower priority than many of the bugs that affect pathauto.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented

Instead of the ctype_alnum check can you try using 

    if (preg_match('/^[a-z0-9]+$/iD', $separator)) {

and see if that works?

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Active » Fixed

I used a slightly modified version which will support international characters well

if (preg_match('/^(\pL|\pN)+$/D', $separator)) {

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Fixed » Postponed

I reverted this because ctype_alnum is more reliable than other solutions. If someone who really cares about this can please come forward with a patch that has been tested on a large number of systems, I'd be happy to commit.

It seems that common systems with differences are:
Fedora Core
*BSD

So any testing should include those.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Postponed » Fixed

Hopefully fixed again - see http://drupal.org/node/167787 again.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)