Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

All uppercase folder name "HTTP" leads to crash of cron #20

Open
rolandjaeger opened this issue Dec 10, 2020 · 1 comment · May be fixed by #21
Open

All uppercase folder name "HTTP" leads to crash of cron #20

rolandjaeger opened this issue Dec 10, 2020 · 1 comment · May be fixed by #21

Comments

@rolandjaeger
Copy link

In my test environment (Ubuntu 18.04) everything is fine but on our managed server at Domainfactory, which is using some custom Gentoo distribution, the cron crashes with the following error:

Uncaught Error: Call to a member function post() on bool in /app/code/community/Mageone/Qps/Model/Cron.php:53

The problem boils down to an apparently very picky handling in the case sensitivity of the Gentoo filesystem (I was too lazy to dig deeper into the exact issue within the autoloader).
In Gentoo, the following line in the constructor returns NULL:
$this->client = Mage::getModel('qps/http_client_curl');

This is because the folder name is all uppercase. Two possible solutions:
Either rename the folder to Http (which is probably the better solution) or change the getModel to $this->client = Mage::getModel('qps/HTTP_client_curl');

@rolandjaeger rolandjaeger changed the title All uppercase folder name "HTTP" leads to crash All uppercase folder name "HTTP" leads to crash of cron Dec 10, 2020
@Schrank
Copy link
Contributor

Schrank commented Dec 15, 2020

Hi @rolandjaeger thanks for telling us. Interesting, this shouldn't work. I'm running all tests on a case-sensitive file system 🤔 - nonetheless you are right, we'll fix it with renaming the directory.

Schrank added a commit that referenced this issue Feb 19, 2021
Fixes #20 All uppercase folder name "HTTP" leads to crash of cron
@Schrank Schrank linked a pull request Feb 19, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants