You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@openoffice.apache.org by bu...@apache.org on 2013/01/16 18:29:53 UTC

[Bug 121633] New: regular expressions fail unless "match case" is also ON

https://issues.apache.org/ooo/show_bug.cgi?id=121633

            Bug ID: 121633
        Issue Type: DEFECT
           Summary: regular expressions fail unless "match case" is also
                    ON
    Classification: Application
           Product: word processor
           Version: AOO 3.4.1
          Hardware: All
                OS: All
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: ui
          Assignee: issues@openoffice.apache.org
          Reporter: jes@martnet.com
                CC: issues@openoffice.apache.org

The new ICU regex engine requires that letter case is significant in the
pattern:
\S is different from \s ("non-space" and "space")
\B is different from \b ("non-boundary" and "boundary")
\p{Uppercase} is valid, \p{uppercase} is not

The "match case" option in Writer interferes with these patterns.

In Writer:
File > New
(type)dt(then)F3 -> dummy text paragraph

Edit > Find & Replace
Search for: \S
Match case: OFF
Options/regular expression: ON
Click: Find

Note that the pattern matches only spaces, where it should match non-space
characters.

Change Match case: ON

Now the pattern works, matching non-space characters.

It seems that the "Match case" option must be on in order to preserve case in
the pattern.

This is not correct, breaking some searches and giving wrong results in others.

The "match case" option should not affect the pattern itself, only how the
match is carried out.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.

[Bug 121633] some regular expressions fail unless "match case" is also ON

Posted by bu...@apache.org.
https://issues.apache.org/ooo/show_bug.cgi?id=121633

Rob Weir <ro...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Assignee|issues@openoffice.apache.or |suchithra75@gmail.com
                   |g                           |

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.

[Bug 121633] some regular expressions fail unless "match case" is also ON

Posted by bu...@apache.org.
https://issues.apache.org/ooo/show_bug.cgi?id=121633

hdu@apache.org <hd...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |ACCEPTED
         Depends on|                            |121880

--- Comment #7 from hdu@apache.org <hd...@apache.org> ---
Yes, the fix for bug 121880 unfortunately reintroduced this problem again.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are watching all bug changes.

[Bug 121633] some regular expressions fail unless "match case" is also ON

Posted by bu...@apache.org.
https://issues.apache.org/ooo/show_bug.cgi?id=121633

hdu@apache.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |CONFIRMED
                 CC|                            |hdu@apache.org
          Component|ui                          |code
             Latest|---                         |AOO 3.4.1
    Confirmation on|                            |
           Assignee|suchithra75@gmail.com       |hdu@apache.org
     Ever confirmed|0                           |1

--- Comment #1 from hdu@apache.org ---
Confirming on AOO 3.4.1 and on todays AOO trunk version.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug 121633] some regular expressions fail unless "match case" is also ON

Posted by bu...@apache.org.
https://issues.apache.org/ooo/show_bug.cgi?id=121633

hdu@apache.org <hd...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|AOO 4.0                     |AOO 4.1

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are watching all bug changes.

[Bug 121633] some regular expressions fail unless "match case" is also ON

Posted by bu...@apache.org.
https://issues.apache.org/ooo/show_bug.cgi?id=121633

--- Comment #4 from SVN Robot <sv...@dev.null.org> ---
"hdu" committed SVN revision 1436886 into trunk:
#i121633# fix ignore-case problem caused by i18nsearch API mess

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug 121633] some regular expressions fail unless "match case" is also ON

Posted by bu...@apache.org.
https://issues.apache.org/ooo/show_bug.cgi?id=121633

DanielAlvaro <jd...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED
                 CC|                            |jdaniel.alvaro@gmail.com

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug 121633] some regular expressions fail unless "match case" is also ON

Posted by bu...@apache.org.
https://issues.apache.org/ooo/show_bug.cgi?id=121633

sebb@apache.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |sebb@apache.org

--- Comment #8 from sebb@apache.org ---
Seems to me that any further fix should include a test case so it cannot happen
yet again.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are watching all bug changes.

[Bug 121633] some regular expressions fail unless "match case" is also ON

Posted by bu...@apache.org.
https://issues.apache.org/ooo/show_bug.cgi?id=121633

--- Comment #5 from Joe Smith <je...@martnet.com> ---
Tested AOO350m1(Build:9611)  -  Rev. 1442201
2013-02-05_04:29:57-Rev.1442466
on Fedora Linux 17

Looks good; problem solved. All my tests I listed above seem to work as
expected now.

Thanks so much for looking into this!

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug 121633] some regular expressions fail unless "match case" is also ON

Posted by bu...@apache.org.
https://issues.apache.org/ooo/show_bug.cgi?id=121633

--- Comment #2 from SVN Robot <sv...@dev.null.org> ---
"hdu" committed SVN revision 1436862 into trunk:
#i121633# fix search for upper-case character classes in ignore-case regular
...

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug 121633] some regular expressions fail unless "match case" is also ON

Posted by bu...@apache.org.
https://issues.apache.org/ooo/show_bug.cgi?id=121633

Joe Smith <je...@martnet.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|CLOSED                      |REOPENED
         Resolution|FIXED                       |---

--- Comment #6 from Joe Smith <je...@martnet.com> ---
I don't know what's happened but this problem is NOT fixed in AOO 4.0 release.

Testing
AOO400m3(Build:9702)  -  Rev. 1503704
2013-07-16 11:18:40 (Tue, 16 Jul 2013) - Linux i686

Edit > Find & Replace
Search for: \Sa

finds only a's preceeded by a space--the pattern calls for non-space.

Tick "Match case" and it works properly.

Maybe the relevant patch did failed to go into the AOO 4 line?

I've changed the status to "REOPENED"; I hope that's right.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are watching all bug changes.

[Bug 121633] some regular expressions fail unless "match case" is also ON

Posted by bu...@apache.org.
https://issues.apache.org/ooo/show_bug.cgi?id=121633

hdu@apache.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|CONFIRMED                   |RESOLVED
         Difficulty|---                         |medium
         Resolution|---                         |FIXED
   Target Milestone|---                         |AOO 4.0

--- Comment #3 from hdu@apache.org ---
Thanks for catching this1 Please check with one of the next nightly builds at
http://ci.apache.org/projects/openoffice/ when they successfully build a
revision >=1436862.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug 121633] some regular expressions fail unless "match case" is also ON

Posted by bu...@apache.org.
https://issues.apache.org/ooo/show_bug.cgi?id=121633

Oliver-Rainer Wittmann <or...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.1.0                       |---

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are watching all bug changes.

[Bug 121633] some regular expressions fail unless "match case" is also ON

Posted by bu...@apache.org.
https://issues.apache.org/ooo/show_bug.cgi?id=121633

--- Comment #9 from Joe Smith <je...@martnet.com> ---
I'm still seeing this problem in dev builds of 4.1

AOO410m14(Build:9760)  -  Rev. 1573613
2014-03-04_02:13:02-Rev.1573839

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are watching all bug changes.

[Bug 121633] some regular expressions fail unless "match case" is also ON

Posted by bu...@apache.org.
https://issues.apache.org/ooo/show_bug.cgi?id=121633

Joe Smith <je...@martnet.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|regular expressions fail    |some regular expressions
                   |unless "match case" is also |fail unless "match case" is
                   |ON                          |also ON

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.