Bug 141538

Summary: UI Remove button does not work in Select Address List dialog of Mail Merge Wizard
Product: LibreOffice Reporter: NISZ LibreOffice Team <libreoffice>
Component: WriterAssignee: Gabor Kelemen (allotropia) <kelemeng>
Status: VERIFIED FIXED    
Severity: normal CC: kelemeng, nemeth, varga.balazs3
Priority: medium Keywords: implementationError
Version: 6.4.0.3 release   
Hardware: All   
OS: All   
See Also: https://bugs.documentfoundation.org/show_bug.cgi?id=113699
Whiteboard: target:7.2.0 target:7.1.4
Crash report or crash signature: Regression By:
Bug Depends on:    
Bug Blocks: 102998    
Attachments: Simple MM datasource file
Mail Merge Wizard before deleting the data source
Mail Merge Wizard after deleting the data source

Description NISZ LibreOffice Team 2021-04-07 13:57:04 UTC
Created attachment 171003 [details]
Simple MM datasource file

The Remove button of the Select Address List dialog of the Mail Merge Wizard does not work on non-GTK backends.

Steps to reproduce:
    1. Tools – Mail Merge Wizard
    2. Go to the 3. Insert Address Block panel
    3. Press the Select Address List button
    4. Add a data source such as the attached file, Press OK
    5. The Insert Address Block panel displays the Current address list below the Select Address List button.
    6. Press the Select Address List button again, press Yes in the confirmation dialog.
    7. The data source disappears from the Select Address List dialog as expected. Press OK.

Actual results:
The Insert Address Block panel still displays the Current address list below the Select Address List button. 
Reopening the Select Address List dialog – as well as the View – Data Sources pane – still displays the data source as existing.

Expected results:
The data source should be removed for good. 
This happens correctly on Linux with GTK3 backend, but not on Windows or Linux + QT5 backend.

LibreOffice details:
Version: 7.2.0.0.alpha0+ (x64) / LibreOffice Community
Build ID: 465b8b0e9ad4b0c9c7701dee2820a99c5d00b5bf
CPU threads: 4; OS: Windows 10.0 Build 18363; UI render: Skia/Raster; VCL: win
Locale: en-US (hu_HU); UI: en-US
Calc: CL
Comment 1 NISZ LibreOffice Team 2021-04-07 13:57:26 UTC
Created attachment 171004 [details]
Mail Merge Wizard before deleting the data source
Comment 2 NISZ LibreOffice Team 2021-04-07 13:57:55 UTC
Created attachment 171005 [details]
Mail Merge Wizard after deleting the data source

Reopen this dialog and the data source comes back.
Comment 3 Gabor Kelemen (allotropia) 2021-04-07 15:47:32 UTC
Proposed patch: https://gerrit.libreoffice.org/c/core/+/113747
Comment 4 Commit Notification 2021-04-16 09:16:27 UTC
Gabor Kelemen committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/a258d42c77c3ce65a9455d08dc6106310222bc7b

tdf#141538 Make Remove button work in Select Address List dialog

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 5 Commit Notification 2021-04-20 19:41:15 UTC
Gabor Kelemen committed a patch related to this issue.
It has been pushed to "libreoffice-7-1":

https://git.libreoffice.org/core/commit/99ca6b660fd911e9e60b63ae286c588aedfb01d0

tdf#141538 Make Remove button work in Select Address List dialog

It will be available in 7.1.4.

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 6 Commit Notification 2021-04-22 09:00:31 UTC
Caolán McNamara committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/9be6795255f7348ca002b141310a2bd458e91b0e

tdf#141538 default to returning contents of first text column

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 7 NISZ LibreOffice Team 2021-05-05 14:53:05 UTC
Verified in:

Version: 7.2.0.0.alpha0+ (x64) / LibreOffice Community
Build ID: ef78c7d93e4f170ac7a7f0307a8305052524eb53
CPU threads: 4; OS: Windows 10.0 Build 17134; UI render: default; VCL: win
Locale: ug-CN (hu_HU); UI: hu-HU
Calc: threaded