Bug 464876

Summary: Issues adding Environment Variables to flatpak
Product: [Applications] systemsettings Reporter: Shubham Arora <shubhamarora>
Component: kcm_flatpakAssignee: Plasma Bugs List <plasma-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: i6icgp6, joshiesuhaas0, kde, me, nate
Priority: NOR    
Version: 5.26.90   
Target Milestone: ---   
Platform: Arch Linux   
OS: Linux   
Latest Commit: Version Fixed In: 5.27.5
Attachments: Steps of Reproduction
Steps of Reproduction

Description Shubham Arora 2023-01-26 21:32:23 UTC
Created attachment 155683 [details]
Steps of Reproduction

SUMMARY/ OBSERVED RESULT
Issue 1: 
1. Add an environment variable. 
2. Try adding a new environment variable. The text within the add new prompt is preserved. 

Issue 2
1. Reopen the flatpak setting after navigating away. 
2. Add another environment variable. The variable is not shown in the UI immediately. 
3. Click apply. 
4. Navigate away and return to the application to see the environment variable successfully added. 

EXPECTED RESULT
Environment variables are immediately Added

SOFTWARE/OS VERSIONS
Operating System: EndeavourOS 
KDE Plasma Version: 5.26.90
KDE Frameworks Version: 5.102.0
Qt Version: 5.15.8
Kernel Version: 6.1.8-arch1-1 (64-bit)
Graphics Platform: Wayland
Processors: 24 × AMD Ryzen 9 3900X 12-Core Processor
Memory: 15.5 GiB of RAM
Graphics Processor: AMD Radeon RX 580 Series
Manufacturer: Gigabyte Technology Co., Ltd.
Product Name: X570 AORUS ELITE
Comment 1 David Edmundson 2023-01-26 21:36:23 UTC
Is this the right attachment? It appears to show tiling issues
Comment 2 Shubham Arora 2023-01-26 21:42:12 UTC
Created attachment 155684 [details]
Steps of Reproduction

Upload correct video.
Comment 3 ratijas 2023-04-20 13:27:54 UTC
There were multiple bugs you hit at once.

The one that new entries might not show up until you switch apps back and forth is due to a wrong usage of model signals, fixed via https://invent.kde.org/plasma/flatpak-kcm/-/merge_requests/91

The other one that lets users enter the same variable name twice was fixed by introducing validators in the model and (partially) in the UI by disabling "OK" button when input is invalid, via https://invent.kde.org/plasma/flatpak-kcm/-/merge_requests/107

The same MR introduces new logic to clear input before the dialog is shows, so you don't get previous values still there. However, another MR, https://invent.kde.org/plasma/flatpak-kcm/-/merge_requests/109, is going to make those dialogs even better and create them on demand, so no "clear text" hacks would be needed anymore.
Comment 4 ratijas 2023-04-22 22:10:13 UTC
*** Bug 468804 has been marked as a duplicate of this bug. ***