[ZBX-7459] incorrect regexp saving and testing Created: 2013 Nov 29 Updated: 2017 May 30 Resolved: 2014 Jul 09 |
|
Status: | Closed |
Project: | ZABBIX BUGS AND ISSUES |
Component/s: | API (A), Frontend (F) |
Affects Version/s: | 2.2.0 |
Fix Version/s: | 2.3.2 |
Type: | Incident report | Priority: | Critical |
Reporter: | richlv | Assignee: | Martins Valkovskis |
Resolution: | Fixed | Votes: | 0 |
Labels: | globalregexps, regexps | ||
Remaining Estimate: | Not Specified | ||
Time Spent: | Not Specified | ||
Original Estimate: | Not Specified |
Attachments: | global_regexps.png regression.png | ||||
Issue Links: |
|
Description |
-------- testcase 1 create new global regexp, regexp " 1" and test string " 1" (without quotes). observe how editing the regexp still has the leading space. -------- testcase 2 create global regexp "^ 1" and test string " 1". test it. notice how it fails. should not. |
Comments |
Comment by Oleksii Zagorskyi [ 2013 Nov 30 ] |
If you will touch it, then see also |
Comment by Andrejs Čirkovs (Inactive) [ 2014 Jun 10 ] |
RESOLVED in r46337. |
Comment by Ivo Kurzemnieks [ 2014 Jun 25 ] |
(1) In regular expressions list view the column "Expressions" should display data with "pre" class also. Otherwise everything is clumped together. andrewtch RESOLVED by r46818. iivs REOPENED. andrewtch RESOLVED by r46864. Please note that I've also added new class, 'break-lines' for the cases if we have really brutal regular expression without spaces. Also, I've removed repeated definition of td.pre since we already have global .pre class. iivs CLOSED. |
Comment by Ivo Kurzemnieks [ 2014 Jun 26 ] |
TESTED. |
Comment by Andrejs Čirkovs (Inactive) [ 2014 Jun 26 ] |
CLOSED in pre-2.3.2 r46872. Martins, please document this. martins-v Mentioned in https://www.zabbix.com/documentation/2.4/manual/introduction/whatsnew240#miscellaneous_improvements |
Comment by Oleg Egorov (Inactive) [ 2014 Jul 03 ] |
(2) Regression See regression.png andrewtch RESOLVED in development branch svn://svn.zabbix.com/branches/dev/ZBX-7459 r47030. oleg.egorov CLOSED |
Comment by Andrejs Čirkovs (Inactive) [ 2014 Jul 04 ] |
CLOSED, fixed in pre2.3.2 r47046. |
Comment by Pavels Jelisejevs (Inactive) [ 2014 Jul 07 ] |
(3) It would be great to document that slashed are interpreted as literal symbols, not delimiters. martins-v Added to https://www.zabbix.com/documentation/2.4/manual/introduction/whatsnew240#miscellaneous_improvements (see last bulletpoint). Please review. andrewtch CLOSED, fine. <richlv> seems to be listed in whatsnew only, which means that it will disappear later - couldn't find it in https://www.zabbix.com/documentation/2.4/manual/regular_expressions martins-v Added to https://www.zabbix.com/documentation/2.4/manual/regular_expressions#configuration as well. RESOLVED. |
Comment by Andrejs Čirkovs (Inactive) [ 2014 Jul 11 ] |
Thanks, CLOSED |