Bug 116413 - Calc: Using Paste Special > Options > Link, but the target cell doesn't change when source is updated
Summary: Calc: Using Paste Special > Options > Link, but the target cell doesn't chang...
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
4.2.0.4 release
Hardware: Other All
: medium normal
Assignee: Roland Kurmann
URL:
Whiteboard: target:7.2.0
Keywords: bibisected, regression
Depends on:
Blocks: Paste-Special
  Show dependency treegraph
 
Reported: 2018-03-15 02:48 UTC by kam3don
Modified: 2021-05-25 21:22 UTC (History)
4 users (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description kam3don 2018-03-15 02:48:56 UTC
Description:
In D23 I type the number "2". In E23 I also type the number "2".

I copy D23, select E23 and Paste Special > Paste All 
+ Operations: Add 
+ Options: Link.
+ Shift Cells: Don't shift

The number "4" now appears in E23, and the cell reference says =2+($SpreadsheetName.$D$23).

Looks great.

I change the value in D23 to 3, hit Enter, hit Save, but E23 remains the same. 

Steps to Reproduce:
1.Enter a number in a cell.
2.Enter another number in a different cell.
3.Copy the value in the first cell, then select the second cell to paste into.
4.Chose Paste Special > Paste All + Operations Add + Options > Link.
5. Then update the value in the first cell. Hit Enter, hit Save.


Actual Results:  
Instead of updating to the new value as shown in the cell reference formula, the second cell stays at the same value.

Expected Results:
The second cell should have updated its value to reflect the changed value of the first cell, to which it is linked.


Reproducible: Always


User Profile Reset: Yes


OpenGL enabled: Yes

Additional Info:
From the LibreOffice Help Files (Paste Special)

Inserts the contents of the clipboard into the current file in a format that you can specify.

Choose Edit - Paste Special

Operations 
Select the operation to apply when you paste cells into your sheet. 

Add 
Adds the values in the clipboard cells to the values in the target cells. Also, if the clipboard only contains comments, adds the comments to the target cells. 

Options 
Sets the paste options for the clipboard contents. 

Link 
Inserts the cell range as a link, so that changes made to the cells in the source file are updated in the target file. To ensure that changes made to empty cells in the source file are updated in the target file, ensure that the Insert All option is also selected. 

You can also link sheets within the same spreadsheet. When you link to other files, a DDE link is automatically created. A DDE link is inserted as a matrix formula and can only be modified as a whole. 




User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_13_3) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/64.0.3282.186 Safari/537.36
Comment 1 raal 2018-03-15 06:40:39 UTC
I can confirm. Need to recalculate with ctl+shift+F9 to get right value

Version: 6.1.0.0.alpha0+
Build ID: cab04bc39b5164ea74216cd849c3af5f5b298f79
CPU threads: 4; OS: Linux 4.4; UI render: default; VCL: gtk3; 

regression in 4.2: 8e7bade4e7314a340c77edd9042e230f61f0323d is the first bad commit
commit 8e7bade4e7314a340c77edd9042e230f61f0323d
Author: Matthew Francis <mjay.francis@gmail.com>
Date:   Sat Sep 5 18:33:36 2015 +0800

    source-hash-4c99a427ee4adaeddb2682c192384bad21d9d09b
    
    Bibisect: This commit covers the following source commit(s) which failed to build
    c7bdee8dbd1cf260a8513a0d31b36f90daa70f1c
    77ec47356025de4e46f48f94629f896349b0a8e5
    e9c5eb60d53204261c7937108bd53e86e46fc2f3
    75dec25730c88bdb8eb5e2a3f92689460fa89d29
    c008dc483f8c6840803983e7e351cec6fdd32070
    46419cd7a2d453c6f252c28dfb9dbfb08605e1c4
    dcf04c58456d8285bdcaa2921b6c08d48e09dab3
    dabeb3d643c0ebd2cb7fdce333d0e6248b1488ad
    8b252f30267d043522ff2cbf2bf42275bb7a6ec6
    76ca1529edb9afdb1838ab9a9e01fa231148c1d8
    341a4c87e977a2fd3948de6eaa647be4b32e6ebc
    3b3b0c04385851f120dc26d26e40f0d1c6344274
    cb4a47887df282196c9aa529269d5115306813c1
    1fe76b401760e0b31095d0a4d7e877aeee1624dc
    6ea53929245eb496d6954ab266636978653f1784
    65be1e27254ff1292b3593af42fbfb0235c26fcd
    b6a6a26698cb56a194c7c99b0ad267e60e3a05d5
    3b0c069c9a157c4cd9ec5636c776115af6d9664f
    458df361e0761651b6d17f4fb730f996243bc9f3
    2a5ea9ee0ba2b38d1810828a9fdf884d9b0fd198
    e3b91687590f08438b5a5d4eec72e634b11a8589
    57538e5f5002ecc2df95087244cb0431867c443e
    df90b9eaa16a52d076658ffc2fd1f65f12d1a622
    2a1c5aba7640416c78501116dd42d12e74fe4734
    66d3f24334e69e1655e83520950c59a0bda095a3
    8a39b8ce354bd42325ff61c07cfdc7a150d2925a
    91f7e9e02e72b46c881656a6b493fac276a6822b
    33a417fc82ce3c5f1b3d435d912eb4b5afc52054
    7a522da4f3946006fd325d498845225b8a8836f2
    5f188d659e8601e577f3a837c9dd3459761371ac
    92a78a052efcb3122932894fb446c62733daad42
    9186ae043b7e8afa5730b9d70818360cdfef2201
    359f33c5d0d39c4fcc57cba199a0d5b9a66c8fb9
    b1391232450af9e81079a2afe2173c422c8e9e3b
    e639e3068c30bb614c394466f41fa450ee8b2dbb
    cf02151987b19b12ca0be463732765bd35f54028
    2c92a92e2fa2503f0381b89f316f982eda580b6e
    21a1bce4301d3d8de9702373c830d2e115223991
    878f46727d8bcf1f75d056d9270ef3e2fe0b9d88
    bb7d5ce2a8bd1dca51eb627aa2df811541053969
    4347e3b15f10784b482544bd6324d3fcd4f0146c
    ec0080c40cfdb26896537f47a4c2e0439f9afdb1
    
    commit 4c99a427ee4adaeddb2682c192384bad21d9d09b
    Author:     Kohei Yoshida <kohei.yoshida@gmail.com>
    AuthorDate: Mon Jun 24 16:19:02 2013 -0400
    Commit:     Kohei Yoshida <kohei.yoshida@gmail.com>
    CommitDate: Mon Jun 24 16:51:45 2013 -0400
    
        Fix incorrect merge.
    
        Change-Id: I1337413e1ee49b7d90671ac517dbb2bd918dbebf
Comment 2 Cor Nouws 2019-02-26 21:24:33 UTC
still an issue in Version: 6.3.0.0.alpha0+
Build ID: a798926550ab3ea8613f28a7be2672ebe9e7dfbe
CPU threads: 4; OS: Linux 4.15; UI render: GL; VCL: gtk3; 
TinderBox: Linux-rpm_deb-x86_64@86-TDF, Branch:master, Time: 2019-02-24_03:53:22
Locale: nl-NL (nl_NL.UTF-8); UI-Language: en-US
Calc: threaded
Comment 3 QA Administrators 2021-02-26 04:11:13 UTC Comment hidden (obsolete)
Comment 4 Roland Kurmann 2021-04-04 19:24:06 UTC
Bug is still present in 

Version: 7.2.0.0.alpha0+ / LibreOffice Community
Build ID: f1b55d3f8e963069fc798bcf559ae9af2bf18b64
CPU threads: 4; OS: Linux 5.3; UI render: default; VCL: x11
Locale: en-US (en_US.UTF-8); UI: en-US
Calc: threaded

I guess the registration as a listener is missing.
Comment 5 Roland Kurmann 2021-04-04 19:44:43 UTC
After saving, closing and opening the spreadsheet, the second cell is immediately updated. Then it works as expected.

So, closing and reopening the spreadsheet is a workaround.
Comment 6 Roland Kurmann 2021-04-25 05:02:48 UTC
I'll soon submit a patch.
Comment 7 Commit Notification 2021-05-21 14:59:53 UTC
scito committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/2eb95408912f2292e2c7e8634c628a29f6c241b7

tdf#116413 after Paste Special with ADD and As Link attach dest cells

It will be available in 7.2.0.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.
Comment 8 Commit Notification 2021-05-25 21:19:18 UTC
scito committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/0563867d60ad477235bb5595c895a4f1e06a53b1

tdf#116413 keep AttachNewFormulaCell methods together

It will be available in 7.2.0.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.