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

Wrong requirements for installation #896

Closed
lucamaro opened this issue Nov 5, 2014 · 5 comments
Closed

Wrong requirements for installation #896

lucamaro opened this issue Nov 5, 2014 · 5 comments
Labels

Comments

@lucamaro
Copy link

lucamaro commented Nov 5, 2014

Please update the php min version requirement: ternary operator "?:" is supported since php 5.3 and not 4.3. (see [http://php.net/manual/en/language.operators.comparison.php]).

Thanks.
Luca

@protich
Copy link
Member

protich commented Nov 5, 2014

Where do we have the wrong min requirement listed?

@lucamaro
Copy link
Author

lucamaro commented Nov 5, 2014

There is a reference in setup/inc/install-prereq.inc.php, but probably even some code that performs the check. Today I was able to finish successfully the installation (on an old linux machine...), but not able to open login page.

@protich
Copy link
Member

protich commented Nov 5, 2014

What version did you install? Latest version of osTicket requires PHP 5.3+

@lucamaro
Copy link
Author

lucamaro commented Nov 7, 2014

On our server we have PHP version 5.2.5 and a working installation of osticket 1.7 downloaded from the osticket site. There are two problems in my opinion: the first is that setup process should block installation when php < 5.3 is found. The second: just updating sources in 1.7 branch may break compatibility with your php.

I have no problem in updating my outdated php, so feel free to close the issue. And thanks for your attention.

@protich
Copy link
Member

protich commented Nov 7, 2014

Thanks for the feedback - I just wanted to get to the bottom of it. I think some of the bug fixes we back ported to 1.7 didn't take into consideration the PHP version requirements. We'll address the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants