https://bz.apache.org/bugzilla/show_bug.cgi?id=67442

--- Comment #8 from Jakub <[email protected]> ---
OK, I see. The fact that these tests fail is a prove that the bug is there. The
last patch has the new method which fixes the bug named "onDeleteFormula_new".
The old "onDeleteFormula" was still left there and was used by the tests.
That's why they failed. I had that to double check that issue is reproducible
and included it in the patch by mistake... Quickest would be if you just remove
the "onDeleteFormula" and rename the "onDeleteFormula_new" to
"onDeleteFormula". Or I can give you another patch where it will be done if
that suits you better.

Please note that I believe that the change I'm suggesting is really fixing the
onDeleteFormula logic which clearly is wrong and it is likely to cause other
problems in similar scenarios (i.e. removing a cell which contains a master
formula referenced by a 2D range). I too have spent quite a lot of time on the
issue. Setting up a local POI environment also wasn't as smooth as I'd like to
(the gradle plugin in eclipse didn't work well :-( ). So I hope that my effort
was not in vain and I will be able to  further help you in improving POI.

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to