Reviewing merge request #16: Bug for escaping config values + timeout of 0 not recognized

Escaping of the configuration file values was wrong, it would escape ', which shouldn't have been escaped. I've created a escape_ini to escape values correctly (str_replace " with \").

Secondly, a execution timeout value of 0 (infinite) wasn't recognized as such, so now the code reads > 60, or 0 to be valid.

Commits that would be merged:

Version 1
  • Version 1
  • 49a1757
  • 3af6e0b
  • Bug where configuration file was escaped incorrectly

Showing 49a1757-3af6e0b

Comments

Pushed new version 1

→ State changed from Open to Closed

Merged.

Add a new comment:

Login or create an account to post a comment

How to apply this merge request to your repository