[REVIEW] don't try to update range names twice in ScDocument::TransferTab

Markus Mohrhard <markus.mohrhard -AT- googlemail.com>
Thu, 1 Mar 2012 12:50:46 +0100

Hey,

the attached patch prevents double updating of the index for copied range names.

The new code updating range names is in cell.cxx adjustRangeName which
does the job directly in the formula tokens. The removed code did
something similar for global range names but was incomplete and
contained some errors. In some cases the removed code did stilla djust
the range name while in most cases it did not.

There is no need for this code anymore since all this functionality is
now covered by adjustRangeName and the old code could not deal with
local range names and some more advanced adjustments while copying
between different documents.

Regards,
Markus

Context

[REVIEW] don't try to update range names twice in ScDocument::TransferTab · Markus Mohrhard
- Re: [REVIEW] don't try to update range names twice in ScDocument::TransferTab · Markus Mohrhard
  - Re: [REVIEWED][3-5] don't try to update range names twice in ScDocument::TransferTab · Kohei Yoshida
    - Re: [PUSHED][3-5] don't try to update range names twice in ScDocument::TransferTab · Petr Mladek

Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.