search

cmb69 / pagemanager_xh

Page management for CMSimple_XH
GNU General Public License v3.0
4 stars 3 forks source link

error with use reserved characters #64

Closed olape-git closed 4 years ago

olape-git commented 4 years ago

Many problems because reserved characters are not excluded or replaced. In case of an update it does not come into effect, so it does not cause any problems and in case of a new installation it would be possible.

https://github.com/cmsimple-xh/cmsimple-xh/issues/413

If all reserved characters are replaced with $tx['urichar']['org'], $tx['urichar']['new'], then under certain situations the page manager will delete the complete content of the content.html.

https://github.com/cmb69/pagemanager_xh/issues/63

cmb69 commented 4 years ago

Danke für den Pull-Request! Im Prinzip geht das so, aber es gibt wohl noch ein paar weitere Zeichen, die Probleme verursachen können, so dass mir preg_quote() zunächst die einfachere Lösung scheint. Daher habe ich den Bug per 09f532ada3fe111d2bd2e7d9c7694458855ba539 gefixt.

Nebenbei: wenn du in der Commit-Message des Pull-Requests fix #63, fixes #63 oder closes #63 schreibst (es gibt noch ein paar weitere Varianten, die Github versteht), dann wird der PR automtisch mit dem zugehörigen Issue verknüpft.

olape-git commented 4 years ago

Ja, theoretisch gibt es weiter Zeichen, aber diese haben keine Probleme verursacht. Aber ist auch egal. Wichtig ist, es ist erledigt.

Das mit fix, fixes, closes versuch ich mir zu merken.