Comparer.php Fails with special characters

[joshwegener]

Having issues when comparing special characters...

// file ./concrete/vendor/punic/punic/code/Comparer.php
   // .....
    public function compare($a, $b)
       // .....
       // failing when comparing "Latvia" and "St. Barthélemy"
       $result = $this->caseSensitive ? strnatcmp($a, $b) : strnatcasecmp($a, $b)
       // .....

[Remo]

can you please elaborate this a bit more? It works fine for me, not matter if iconv and/or Collator is missing. I'm running \Punic\Territory::getList('c') and the list is always properly sorted. Thanks

[joshwegener]

You are welcome to come onto my server and have a look. I'm installing concrete5, and when it gets to \Punic\Territory::getList('c') it crashes. The server get's a internal server error 500.

I have tracked it down line by line, and the 2 values listed are the values being passed in and that is the line it's failing on.

[mlocati]

I tested the line you reported and everything seems to work fine for any PHP version: see https://3v4l.org/Vt0Ji

What's your concrete5 version? What's your current language?

[joshwegener]

Concrete5.8.0.3 and using English. I wonder if it's some sort of configuration issue... it's a centos server running cpanel. My PC had a CPU failure yesterday, I will try and get a PHP info page up so you can see thoes details. Only thing to note is that the previous version of Concrete5 works fine on that server.

…

On Jan 21, 2017 11:07 AM, "Michele Locati" ***@***.***> wrote: I tested the line you reported and everything seems to work fine for any PHP version: see https://3v4l.org/Vt0Ji What's your concrete5 version? What's your current language? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#88 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABrgCSrqKsVpBGXWn9DpqkJx2qXZ0Cqzks5rUkl7gaJpZM4Lpkey> .

[joshwegener]

Here is the PHP info page: http://reds.whoisjoshw.com/info.php

And if you just go to the root, you can see the server 500 error that I'm getting on step 3.

[mlocati]

It could be very useful to see the PHP error log, a simple "Error 500" page isn't a great help 😉

[joshwegener]

There is no errors in the log.

[joshwegener]

I would be happy to share the user details for ssh if you would like to check it out... I have spent hours trying to debug this.... JoshWegener@gmail.com - E-Mail me if you want the server details. It's just a VM, so I'm not worried about you messing it up...

[mlocati]

Maybe there could be something in the apache error log...

[mlocati]

I would be happy to share the user details for ssh if you would like to check it out... I have spent hours trying to debug this.... JoshWegener@gmail.com

I'm very curious... I'll write you an email

[joshwegener]

I didn't see anything in the apache logs as well.

[joshwegener]

Feel free to change anything in ~/reds.whoisjoshw.com for testing. And if you change that file where I noted above, you can see it is failing there... Just not sure why. I thought it was the é in "Barthélemy" causing some sort of issue; but when I test just that peace, it does appear to work... If you need a fresh copy of the code, just

cd /home/whoisjoshw/reds.whoisjoshw.com && rm -rf ./* && unzip ../concrete5-8.0.3.zip && mv concrete5-8.0.3/* ./ && rmdir concrete5-8.0.3/

[mlocati]

1. I tried to update Punic to the very latest version (as per #4699), without success.
2. I tried to enable the concrete5 debug errors (with concrete/bin/concrete5 c5:config -g set concrete.debug.detail debug and concrete/bin/concrete5 c5:config -g set concrete.debug.display_errors true), but nothing changed.
3. I checked the apache error log, but I found no error

[joshwegener]

I have no idea what is causing this error.... or the lack of an error... other then server 500

[mlocati]

I also tried to manually enable the PHP error log by placing these lines here and there:

ini_set('display_errors', 1);
error_reporting(-1);
ini_set('error_log', '/path/application/files/php-errors.log');
ini_set('log_errors', 1);

but still no success

[mlocati]

So, the problem was not with Punic, isn't it?

[joshwegener]

It appears it had something to do with the cpanel server configuration and modules. I rebuild it with easy apache 4 and it fixed the issue.

[mlocati]

@joshwegener This has been very hard to debug, but I finally solved the underlying problem... Yes, the source of the problem was Punic, that under very specific cases broke the execution.
See #89

[joshwegener]

wow

…

On Fri, Feb 3, 2017 at 9:07 AM, Michele Locati ***@***.***> wrote: @joshwegener <https://github.com/joshwegener> This has been very hard to debug, but I finally solved the underlying problem... Yes, the source of the problem was Punic, that under very specific cases broke the execution. See #89 <#89> — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#88 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABrgCa_mnug-3LCMXhR7-BDmFJTANRcsks5rY1C8gaJpZM4Lpkey> .

-- Thanks, Josh Wegener