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

Changeable ticks timing and fix an include path issue #98

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

gymadarasz
Copy link

@gymadarasz gymadarasz commented May 17, 2017

Hi,
I really like your useful project however I found two issue when I tried to install at first time. I think not me the first who having same issues so I fix it:
issue 1: PHP can't find the users.php if includes are in a different folder and it caused a php fatal error (more flexible solution is an autoloader for e.g. composer)
issue 2: tick() function called once per seconds but these protected variables able to the extender class to override the tick speed even dynamically

@gymadarasz gymadarasz changed the title Gymadarasz master Changeable ticks timing and fix an include path issue May 17, 2017
gymadarasz added a commit to gymadarasz/PHPWebsockets that referenced this pull request Apr 10, 2018
issue 1: PHP can't find the users.php if includes are in a different folder and it caused a php fatal error (more flexible solution is an autoloader for e.g. composer)
issue 2: tick() function called once per seconds but these protected variables able to the extender class to override the tick speed even dynamically

See in original PR: ghedipunk#98
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 this pull request may close these issues.

1 participant