Bug 97497 - Autocorrect Replace term works only every second time or not at all
Summary: Autocorrect Replace term works only every second time or not at all
Status: RESOLVED WORKSFORME
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
5.0.4.2 release
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: haveBacktrace
Depends on:
Blocks:
 
Reported: 2016-02-02 08:11 UTC by Daimonion
Modified: 2017-05-12 06:16 UTC (History)
1 user (show)

See Also:
Crash report or crash signature:


Attachments
autocorrect dialog with affected replace line marked (43.53 KB, image/png)
2016-02-02 08:11 UTC, Daimonion
Details
Backtrace of assertion failure 5.2 (11.37 KB, text/plain)
2016-02-10 12:06 UTC, Buovjaga
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Daimonion 2016-02-02 08:11:19 UTC
Created attachment 122324 [details]
autocorrect dialog with affected replace line marked

Hey Guys

I made a new automatic replace shortcut to replace every ,, with :
I want this to insert times in cells easier. 

After finding the correct format for these shortcut

.*,,.* 

Calc replaces the inserted value 15,,55 only every second time to 15:55. The first time the value is not replaced and the cell contains 15,,55 as value.


So, i think replacing does not work at all time and want to file these with this report. 


If i should do anything or test anything to clarify the situation please let me know. I will watch this bug.
Comment 1 Buovjaga 2016-02-10 12:06:47 UTC
Created attachment 122495 [details]
Backtrace of assertion failure 5.2

I get this the first time I input 15,,55 after defining the rule. I have to delete and redefine to reproduce it.

Win 7 Pro 64-bit Version: 5.2.0.0.alpha0+
Build ID: 76ec54e8c9f3580450bca85236a4f5af0c328588
CPU Threads: 4; OS Version: Windows 6.1; UI Render: default; 
TinderBox: Win-x86@39, Branch:master, Time: 2016-02-09_00:10:35
Locale: fi-FI (fi_FI)
Comment 2 Buovjaga 2016-02-10 12:07:41 UTC
Cannot reproduce problem, but perhaps it is related to the assertion failure I saw with 5.2.

Win 7 Pro 64-bit, Version: 5.0.4.2 (x64)
Build ID: 2b9802c1994aa0b7dc6079e128979269cf95bc78
Locale: fi-FI (fi_FI)

Version: 5.2.0.0.alpha0+
Build ID: 76ec54e8c9f3580450bca85236a4f5af0c328588
CPU Threads: 4; OS Version: Windows 6.1; UI Render: default; 
TinderBox: Win-x86@39, Branch:master, Time: 2016-02-09_00:10:35
Locale: fi-FI (fi_FI)
Comment 3 Daimonion 2016-02-10 12:54:00 UTC
I tried this today, because you couldn't reproduce it on 5.0.4.2

My Installation:

Version: 5.0.4.2 (x64)
Build-ID: 2b9802c1994aa0b7dc6079e128979269cf95bc78
Gebietsschema: de-DE (de_DE)

I have to correct me. replacing does not work every second time. 
With todays test it works for every time which was inserted before at least once. (not 100% sure about the fact that it hast to be inserted at least once but at the moment it was the best hit)

I had to change cell format to [HH]:MM to produce the bug.
With cell format [HH]:MM:SS everything was fine and replacement works as expected. 

Maybe this gives you a better hint to find the failure. If you want, i can make a video about this. 


Regards
Daimonion
Comment 4 Buovjaga 2016-02-10 13:01:23 UTC
(In reply to Daimonion from comment #3)
> I had to change cell format to [HH]:MM to produce the bug.
> With cell format [HH]:MM:SS everything was fine and replacement works as
> expected. 

Ok, I just tried with HH:MM, but it worked.

You can create a video, if you wish.
Comment 5 Daimonion 2016-02-10 13:12:42 UTC
Okay, for now it's even not connected with cell formatting type.

Please see this video here:

https://goo.gl/photos/dq6TXgbviStxQWYD6
Comment 6 Daimonion 2016-08-30 06:12:22 UTC
5.1.4.2 x64 Version has this Bug.

Please devs, can you have a look at it?
Comment 7 Carlos 2017-04-04 18:38:09 UTC
I can confirm that the bug is present in
Version: 5.3.1.2 (x64)
Build ID: e80a0e0fd1875e1696614d24c32df0f95f03deb2
CPU Threads: 4; OS Version: Windows 6.19; UI Render: default; Layout Engine: new; 
Locale: en-US (en_US); Calc: group

For me the new autocorrect entry
Replace  .*,,.* with :
doesn't work at all. 

Other replacement work fine. 

The example of using autocorrect to to replace 
.*,,.* 
with
:
is even mentioned in Help
https://help.libreoffice.org/index.php?title=Common/Replace&Language=en-US&System=WIN&Version=5.3#bm_id3154810
Comment 8 Daimonion 2017-05-12 06:16:17 UTC
For me this bug seems to be fixed in Version: 5.3.2.2 (x64)

since i'm using this version i didn't get the described behaviour.

Thanks for fixing.