Conversation
Fix bug hoaproject#90
1 similar comment
| str_replace( | ||
| ['\'', '\\\\'], | ||
| ['\\\'', '\\\\\\'], | ||
| ['\\\'', '\\\\\\\\'], |
There was a problem hiding this comment.
It seems like dumping of these values must be done via var_export(..., true). Right now it's unsafe and unpredictable.
There was a problem hiding this comment.
let's fix the bug within the current mechanism first because it's an easy merge. You can open a PR where var_export is used
There was a problem hiding this comment.
Actually, using var_export is not a bad idea at all. And it will simplify the code. Are you willing to try to update with var_export please?
| str_replace( | ||
| ['\'', '\\\\'], | ||
| ['\\\'', '\\\\\\'], | ||
| ['\\\'', '\\\\\\\\'], |
There was a problem hiding this comment.
Actually, using var_export is not a bad idea at all. And it will simplify the code. Are you willing to try to update with var_export please?
|
@unkind Check this command https://github.com/hoaproject/Devtools#expandflexentities to help you. |
|
@Hywan I have to find out the real purpose over |
|
@unkind You can use |

Fix bug #90