[Upstream] Undo does not preserve conditional formatting

Problem description: I have a spreadsheet with one cell (with an =IF(condition,"CORRECT","INCORRECT") function having one format dependent on its own text value (or else its default format, making it go red or green). Another column of cells each have formatting (Blue Green or Red background) dependent on whether their value is greater, equal or less than the value three cells to the right. Saving as ODF (or opening a Libre Office 3.4-created file with this formatting) the column of cells have only background formatting and no Conditional Formatting shows up while the single condition has correct formatting.

Also if re-formatted as Conditional, then you Format Paint this format onto the same cell dragging down one cell too far, hitting undo to try again will wipe the original conditional format and there's no redo available. Similarly, hitting the Delete key (or Delete Contents without deleting formatting) has the same effect.

Steps to reproduce:
1. ....
2. ....
3. ....

Current behavior:

Expected behavior: Same as Libre Office 3.4 - Conditional Format is retained.

Platform (if different from the browser):

Browser: Opera/9.80 (Windows NT 6.1; Edition Campaign 21) Presto/2.12.388 Version/12.10

Hi rh,

thanks for your report.
Could you pls try the following:
- try 3.6.4 release candidate 1
  http://www.libreoffice.org/download/pre-releases/

- try a daily build
  http://dev-builds.libreoffice.org/daily/

If the issue is not fixed in those, pls attach an example file?

thanks - Cor

Created attachment 70378
Example balance sheet / cash till float spreadsheet

Thanks. I tried version 3.6.4RC1 and after my PC hung on me mid-install leaving a 'program' folder with no access and no owner which I cannot delete or take ownership of even as administrator, I managed to install it in a different folder. (Windows 7 Professional 32 bit on AMD Turion 64-bit processor with 4GB RAM)

Still have some problems, so I attach Cash_Check_test.ods saved using that version of LibreOffice Calc

Cells K2 to K13 have three-way conditional formatting depending on whether lower, equal or higher than cell N2 to N13.

For example, change K4 from 30 to 40 and notice it becomes light-blue. Change it to 20 and it becomes light-red (pink). Set it back to 30 and it becomes green.

Now, still on K4, hit Delete. It's treated as zero, and goes light red.

Hit Undo, and it reverts to 30 and green.

Now select two cells, K4:K5 and hit delete. This time all formatting is lost.

Not to worry, you might think, just hit Undo. The values return.

But, Oh, where has my conditional formatting gone!?

It seems to lose the Conditional Formatting when deleting a multiple selection and the Undo doesn't realise it has gone, so I don't think it's an intended feature of hitting delete on a multi-cell selection.

There doesn't seem to be a problem with Paste Special / Numbers changing the formats now, but at least I can demonstrate a bug of the sort I mentioned with steps to try to replicate it.

Thanks,

rh

I've tried the Version 3.6.4.1 (Build ID: a9a0717) on MacOS 10.7.5.
I still lose the Conditional Formatting when deleting a multiple selection.
See attachment

Created attachment 70380
LibO files with bug using conditional formatting

Undo/Redo fixed in master for 4.0

Fixed the autofill problem in master.

Sorry marked the wrong bug as fixed.

This problem can be reproduced easily with "SampleSource.ods" in test kit Attachment 68944 of Bug 56316.

Steps how to reproduce:
0. download / Unzip Attachment 68944
1. Open "SampleSource.ods" from Test kit
2. With menu 'Format -> Conditional Formatting -> Manage' you see that there
   is some CF for Cell I5
3. Click row heading for row 3
  > row 3 selected
4. <control+c> for copy
5. Click row heading for row 5
6. <control+v> for paste
   > only empty cells in row 5
7. With menu 'Format -> Conditional Formatting -> Manage' you see that there
   is NO CF for Cell I5
8. Click the "Undo" Icon in the Standard Toolbar
   > Contents of row 5 reappears
9. With menu 'Format -> Conditional Formatting -> Manage' you see
   BUG: still NO CF for Cell I5 !!!

Other possibility to reproduce the bug (as observed in <https://bugs.freedesktop.org/show_bug.cgi?id=56316#c13>:
11. Open Attachment 70975 Details for Bug 56316
12. Click common rows / Columns heading at top left corner
    all cells become selected
13. <control+c> for copy
14. Click A1
15. <control+v> for paste
    > Complete sheet contents of sheet sheet "replaced by itself"
      But Conditional Formatting is lost
16. Click the "Undo" Icon in the Standard Toolbar
    > a lot of flickering seems to restore the old contents
    BUG: still NO CF (cells with contents "8" without
         blue background !!!

Already [Reproducible] with
- Server Installation of "LibreOffice 3.6.2.1 rc English UI/ German Locale [Build-ID: ba822cc] on German WIN7 Home Premium (64bit) and Sample of
Step 1, Sample of step 11 crashes this version.

This still worked fine with
- Server Installation of "LibreOffice 3.6.0.4 English UI/ German Locale [Build-ID: 932b512] on German WIN7 Home Premium (64bit)
- Server Installation of "LibreOffice 3.6.0.0.beta2 German UI/Locale [Build-ID: f010139] on German WIN7 Home Premium (64bit) (already new CF dialog, but without "Manage" option

Works for me with both examples in this comment and with parallel installation of "LOdev 4.0.0.0.alpha1+ - ENGLISH UI / German Locale [Build ID: 6aabe09ac092c51d4b394bde9c7ea0055b952e3)]" {tinderbox: Win-x86@6, pull time 2012-11-26 00:29:34} on German WIN7 Home Premium (64bit) with own separate User Profile

@Markus:
We will need a backport to 3.6.

3.6.2 (?) regression

In Version 4.0.0.0.alpha1+ (Build ID: e14b5f33ceae3280b46eb97a88b949d00263956) on Ubuntu 12.04 after copy/paste whole sheet as described in comment 8, the conditional formatting records survives after pasting.
However, new problem arises; on open file in 4.0 original conditional formatting setting (if cell value is > 0, than cell style = Number) is changed to empty value (if cell value is equal = (empty box), then Default style).

Probably another kind of problem.

Libreoffice also still hangs, when there are 12 monthly sheets in a file, when I try to copy/paste content of the whole sheet.

(In reply to comment #10)

>
> Libreoffice also still hangs, when there are 12 monthly sheets in a file,
> when I try to copy/paste content of the whole sheet.

Update! It is not hanging, it just takes several minutes to paste. Few seconds in Libreoffice 3.5.5.

(In reply to comment #11)
> (In reply to comment #10)
>
> >
> > Libreoffice also still hangs, when there are 12 monthly sheets in a file,
> > when I try to copy/paste content of the whole sheet.
>
> Update! It is not hanging, it just takes several minutes to paste. Few
> seconds in Libreoffice 3.5.5.

Can you please open new bug reports for new issues. I'll have a hard time to track bug reports otherwise. As long as the title contains conditional formatting it will show up in my search.

Adding additional bug reports to an existing bug report leads to me loosing track of the different problems mentioned in the bug report.

*** Bug 58509 has been marked as a duplicate of this bug. ***

I can confirm that this issue still exists in 4.0 beta 1.

I've tried the Version 4.0.0.0.beta2 on MacOS 10.7.5.
I still lose the Conditional Formatting when deleting a multiple selection.

Reproducible in:
Microsoft Windows Vista Business x86 6.0.6002 Service Pack 2 Build 6002
Version 4.0.0.1 (Build ID: 527dba6f6e0cfbbc71bd6e7b88a52699bb48799)

lsb_release -rd
Description: Ubuntu 12.10
Release: 12.10

apt-cache policy libreoffice-calc
libreoffice-calc:
  Installed: 1:3.6.2~rc2-0ubuntu4
  Candidate: 1:3.6.2~rc2-0ubuntu4
  Version table:
 *** 1:3.6.2~rc2-0ubuntu4 0
        900 http://archive.ubuntu.com/ubuntu/ quantal-updates/main i386 Packages
        100 /var/lib/dpkg/status
     1:3.6.2~rc2-0ubuntu3 0
        500 http://archive.ubuntu.com/ubuntu/ quantal/main i386 Packages

Not reproducible in Version 4.1.0.0.alpha0+ (Build ID: 28b7359e00fec9fd3bfab3c9105cf250c4320a2)
TinderBox: Win-x86@6, Branch:master, Time: 2013-02-02_00:14:06 using either of Rainer Bielefeld's methods.

Can this be marked as resolved?

Reproducible in:
Microsoft Windows Vista Business x86 6.0.6002 Service Pack 2 Build 6002
Version 4.0.0.3 (Build ID: 7545bee9c2a0782548772a21bc84a9dcc583b89)

Using original reporter's attachment https://bugs.freedesktop.org/attachment.cgi?id=70378 highlight cells K4:K5 -> click Delete -> Undo and the Background Color of cells K4:K5 are not restored.

@Christopher M. Penalver
I agree that this is not working with the original attachment (I had misunderstood the instructions earlier).

Tested on Windows XP with:
Version 4.1.0.0.alpha0+ (Build ID: 28b7359e00fec9fd3bfab3c9105cf250c4320a2)
TinderBox: Win-x86@6, Branch:master, Time: 2013-02-02_00:14:06

I've just tested and the bug is still present in
Version 4.0.1.2 (Build ID: 84102822e3d61eb989ddd325abf1ac077904985)

I'd been hopeful when I saw in Release Notes that some other Conditional Formatting bugs had been resolved.

Thanks to the devs for your continuing efforts.

This is still present in 4.0.1.2, I just wanted to report it.
You cannot undo / redo conditional formatting, like it never happened.

We had some reports where we had to learn that problems looking similar had different roots so that a fix did not solve all problems. May be we here have a similar situation, I will do some tests today.

I decided to mark this one as DUP of "Bug 57661 - CONDITIONAL FORMATTING of range lost after deletion the contents of range":

a) reporter's comment 2 matches with But 57661
b) That bug is already Assigned

For any other bug not matching with Bug 57661 please open new Bugs (if not already reported).

If the problem in that new report can be reproduced with an existing Att. Sample document please cite it due to <https://wiki.documentfoundation.org/QA-FAQ#How_to_use_attached_sample_documents_for_multiple_Bug_Reports> instead of attaching it newly.

*** This bug has been marked as a duplicate of bug 57661 ***

Unmarking as duplicate as the problem noted in the Description https://bugs.freedesktop.org/show_bug.cgi?id=57176#c0 is reproducible in:
Microsoft Windows Vista Business x86 6.0.6002 Service Pack 2 Build 6002
Version 4.0.3.3 (Build ID: 0eaa50a932c8f2199a615e1eb30f7ac74279539)

I can no longer reproduce: https://bugs.freedesktop.org/show_bug.cgi?id=57176#c2 on either the 4.0 or 4.1 / master branches.

Isolating and back-porting (if possible) the conditional formatting code to 3.6 seems unlikely to happen before the last 3.6 release (3.6.7), but if some developer wants to do that they are more than welcome to find / push a patch to gerrit for double review.

Christopher - if you can reproduce this with a 4.0.4 pre-release snapshot, can you attach a sample document with easier to reproduce steps like comment#2. Or preferably a set of simple instructions for an absolutely minimal test - with the relevant UI paths to commands highlighted in each case: Format->Character->Foo etc. :-) I found comment#0 somewhat impenetrable myself.

Thanks !

I can still reproduce (in 4.0.2.1 (Build ID: 7e5467ff8f30d821f4fbf69cb2769163eb64c2c)) the problem I was having in #58509 and it doesn't relate to deleting, but editing conditional formatting.

Would it be better if I reopen this bug too?

Created attachment 80735
Simple Conditional Format paste bug demo with instructions in worksheet

New Attachment CFpasteBug.ods
obsoletes attachment 70378.

This is simpler than attachment 70378

It includes instructions to reproduce the bug in cells A5:A13

It shows that the Conditional Format is still present but not displayed: Use Format Paintbrush to paint format onto the same range (effectively forcing a recalculate of the conditions and a refresh of their display) to make it display properly.

Tested with Windows 7 Professional 32-bit, latest Dev beta
Version: 4.1.0.0.beta2
Build ID: 33224f4f11a05cfad2249e812fcc2975fbb61f6

Unreproducible in:
Version: 4.2.0.0.alpha0+
Build ID: 979def88090633bfee0e0445b19999a1dac71ed
Microsoft Windows Vista Business x86 6.0.6002 Service Pack 2 Build 6002

I've re-opened this bug.

(While I'm here, please accept my apologies for using a complicated attachment originally, which has for a while been obsoleted and I've replaced it with the second of the two non-obsolete attachments, the 14.38 KiB one):

The bug still exists in last night's 4.2.0.0.alpha0+ Win-86@6 build if you use that second attachment, the one named
"Simple Conditional Format paste bug demo with instructions in worksheet"
(actual filename = CFpasteBug.ods)

--------
Version: 4.2.0.0.alpha0+
Build ID: 6372309a0d285f4a176dbcded3f18b72220e123
That's the current nightly build from
http://dev-builds.libreoffice.org/daily/master/Win-x86@6/current/
Windows 7 Professional, 32-bit. AMD Turion 64 dual core. 4GB RAM.
--------

You only need to get to step 4 to confirm the bug from these instructions (included also in the spreadsheet)
1. Select range B1:E1
2. Edit/Copy
3. Select cell B2
4. Right click on B2, Paste Only ► Number (click Yes to confirm paste)

The range B2:E2 does not display its conditional formatting (red or green coloured background) as it should, only the underlying bold, centered plain formatting on a "No Fill" background.

The CF is still associated with the range B2:E2 (see Format/Conditional Format/Manage to confirm).
You can easily restore it to be visible using Format Paintbrush to paint the format from B2:E2 back into B2, which acts as a "Refresh" function for the Conditional Format in that range. Calling the Format Paint in this way would be a suitable fix, though it's probably not very efficient.

The same applies using Paste Special... and pasting anything that doesn't include pasting the formatting from the source range to destination range seems to prevent the Conditional Formatting from being displayed.

I think this tells us what is missing from the code. My pure guess is that some function call that was not obviously important was not replicated during a code rewrite that was intended either to reduce Java dependency from the old OpenOffice.org codebase or to allow new and fancy CF such as Color Scales around about the time of LibO 3.6.

After all, why would you need to concern yourself with formatting in the target cells if you're not pasting the formatting from Source to Target. (For normal formatting this is true - not for formatting that's Conditional on the cell value).

I'm familiar enough with the program behaviour in this regard (the edge cases where the bug is or is not present) that I've suggested below how I'd go about trying to trace and fix the bug in principle, albeit that I am not capable of achieving it myself in practice any time soon.

Bug 57661 has been fixed for a while, and had exhibited similar behaviour (when using Delete to clear the values from cells with Conditional Format, the CF did not display and Undo would not fix it either). Perhaps the patches used there can reveal the correct functions to call to refresh/recalculate the Conditional Format display, or that code can itself be made into a function or method we can use for this fix to avoid code duplication.

I have not developed a full software application in 16 years. I've never worked on a team software project, so I would ne...

I think there was a duplicate bug that I've now marked as Resolved and realised that this one was marked REOPENED.

I'll mark this RESOLVED also, though I have since identified another bug on Paste Special... Add... which does not handle blank cells as if they were zero and ends up putting things in the wrong cells. I'll try to find the time to knock up a simple example and post a new bug for that later today.