[plasmashell] [Bug 447558] "Files linked to the current activity" makes desktops of other activities show nothing before restarting plasmashell

2022-06-26 Thread Fushan Wen
https://bugs.kde.org/show_bug.cgi?id=447558

Fushan Wen  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|REOPENED|RESOLVED

--- Comment #10 from Fushan Wen  ---
(In reply to Robby Engelmann from comment #9)
> (In reply to Fushan Wen from comment #5)
> > Git commit 7f09fb161188a4615af00dce23a8356935f42f00 by Fushan Wen.
> > Committed on 18/01/2022 at 10:12.
> > Pushed by fusionfuture into branch 'master'.
> 
> > 
> > https://invent.kde.org/plasma/plasma-desktop/commit/
> > 7f09fb161188a4615af00dce23a8356935f42f00
> 
> Is this MR integrated in the master branch? If yes, the issue is still
> occuring here using the master branch builds in openSUSE Tumbleweed

This is not caused by the commit. The commit actually fixed the bug. Please see
453110

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 447558] "Files linked to the current activity" makes desktops of other activities show nothing before restarting plasmashell

2022-05-03 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=447558

Nate Graham  changed:

   What|Removed |Added

   See Also||https://bugs.kde.org/show_b
   ||ug.cgi?id=453110

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 447558] "Files linked to the current activity" makes desktops of other activities show nothing before restarting plasmashell

2022-05-03 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=447558

Nate Graham  changed:

   What|Removed |Added

 CC||n...@kde.org

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 447558] "Files linked to the current activity" makes desktops of other activities show nothing before restarting plasmashell

2022-05-02 Thread Robby Engelmann
https://bugs.kde.org/show_bug.cgi?id=447558

Robby Engelmann  changed:

   What|Removed |Added

 Resolution|FIXED   |---
 Status|RESOLVED|REOPENED

--- Comment #9 from Robby Engelmann  ---
(In reply to Fushan Wen from comment #5)
> Git commit 7f09fb161188a4615af00dce23a8356935f42f00 by Fushan Wen.
> Committed on 18/01/2022 at 10:12.
> Pushed by fusionfuture into branch 'master'.

> 
> https://invent.kde.org/plasma/plasma-desktop/commit/
> 7f09fb161188a4615af00dce23a8356935f42f00

Is this MR integrated in the master branch? If yes, the issue is still occuring
here using the master branch builds in openSUSE Tumbleweed

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 447558] "Files linked to the current activity" makes desktops of other activities show nothing before restarting plasmashell

2022-04-21 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=447558

Nate Graham  changed:

   What|Removed |Added

 CC||robby.engelmann@r-engelmann
   ||.de

--- Comment #8 from Nate Graham  ---
*** Bug 452816 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 447558] "Files linked to the current activity" makes desktops of other activities show nothing before restarting plasmashell

2022-02-15 Thread Fushan Wen
https://bugs.kde.org/show_bug.cgi?id=447558

Fushan Wen  changed:

   What|Removed |Added

 CC||h...@kde.org
  Component|Desktop Containment |Folder

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 447558] "Files linked to the current activity" makes desktops of other activities show nothing before restarting plasmashell

2022-02-15 Thread Fushan Wen
https://bugs.kde.org/show_bug.cgi?id=447558

Fushan Wen  changed:

   What|Removed |Added

 CC||leviat...@gmx.com

--- Comment #7 from Fushan Wen  ---
*** Bug 450056 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 447558] "Files linked to the current activity" makes desktops of other activities show nothing before restarting plasmashell

2022-02-14 Thread Oleg Solovyov
https://bugs.kde.org/show_bug.cgi?id=447558

Oleg Solovyov  changed:

   What|Removed |Added

 CC||mcp...@altlinux.org

--- Comment #6 from Oleg Solovyov  ---
Reproducible in 5.24: please backport

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 447558] "Files linked to the current activity" makes desktops of other activities show nothing before restarting plasmashell

2022-01-18 Thread Fushan Wen
https://bugs.kde.org/show_bug.cgi?id=447558

Fushan Wen  changed:

   What|Removed |Added

   Version Fixed In||5.25

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 447558] "Files linked to the current activity" makes desktops of other activities show nothing before restarting plasmashell

2022-01-18 Thread Fushan Wen
https://bugs.kde.org/show_bug.cgi?id=447558

Fushan Wen  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|ASSIGNED|RESOLVED
  Latest Commit||https://invent.kde.org/plas
   ||ma/plasma-desktop/commit/7f
   ||09fb161188a4615af00dce23a83
   ||56935f42f00

--- Comment #5 from Fushan Wen  ---
Git commit 7f09fb161188a4615af00dce23a8356935f42f00 by Fushan Wen.
Committed on 18/01/2022 at 10:12.
Pushed by fusionfuture into branch 'master'.

screenmapper: Remember activity ID in screen mapping

This commit adds an extra string to screenmapper to remember the
activity ID corresponding to the screen ID. Now screenmapper can
correctly handle different activites, and desktops of different
activities can correctly show their items.

The problem

When the user sets "Files linked to the current activity" on one
activity (A) and switches to any other activity (B), though the desktop
on activity B is set to "Show Desktop folder",  there is nothing on
the desktop until the user restarts plasmashell.

The cause

In screenmapper.cpp, `removeScreen()` only checkes for the screen ID,
but two activities can share the same screen ID. Then if both the two
activities are set to "Show desktop folder", then in `setUrl()`

1. After the user changes the desktop folder option on one of the two
activities,
   `removeScreen()` will remove all item records related to the screen
   ID and **the old URL (desktop:/)**.
2. But `addScreen()` will only add back item records for the current activity
   because `addScreen()` will directly return if `m_availableScreens`
   already contains the screen ID (which is added from the current activity).
3. Thus items on the desktop of the other activity incorrectly disappears.

The fix

1. When calling `addScreen()` or `removeScreen()` from FolderModel, also
   consider the activity corresponding to the FolderModel, which is the
   current activity when the FolderModel is initialized.
2. Provide a migrator for the old configuration entries because an extra
   record is added.

M  +1-0containments/desktop/plugins/folder/CMakeLists.txt
M  +9-8   
containments/desktop/plugins/folder/autotests/foldermodeltest.cpp
M  +1-0containments/desktop/plugins/folder/autotests/foldermodeltest.h
M  +3-2containments/desktop/plugins/folder/autotests/positionertest.cpp
M  +1-0containments/desktop/plugins/folder/autotests/positionertest.h
M  +48   -47  
containments/desktop/plugins/folder/autotests/screenmappertest.cpp
M  +1-0containments/desktop/plugins/folder/autotests/screenmappertest.h
M  +16   -14   containments/desktop/plugins/folder/foldermodel.cpp
M  +5-0containments/desktop/plugins/folder/foldermodel.h
M  +118  -60   containments/desktop/plugins/folder/screenmapper.cpp
M  +32   -12   containments/desktop/plugins/folder/screenmapper.h

https://invent.kde.org/plasma/plasma-desktop/commit/7f09fb161188a4615af00dce23a8356935f42f00

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 447558] "Files linked to the current activity" makes desktops of other activities show nothing before restarting plasmashell

2022-01-13 Thread Fushan Wen
https://bugs.kde.org/show_bug.cgi?id=447558

Fushan Wen  changed:

   What|Removed |Added

 CC||balint.juh...@jbm.hu

--- Comment #4 from Fushan Wen  ---
*** Bug 447159 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 447558] "Files linked to the current activity" makes desktops of other activities show nothing before restarting plasmashell

2022-01-01 Thread Bug Janitor Service
https://bugs.kde.org/show_bug.cgi?id=447558

Bug Janitor Service  changed:

   What|Removed |Added

 Status|CONFIRMED   |ASSIGNED

--- Comment #3 from Bug Janitor Service  ---
A possibly relevant merge request was started @
https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/775

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 447558] "Files linked to the current activity" makes desktops of other activities show nothing before restarting plasmashell

2021-12-31 Thread Fushan Wen
https://bugs.kde.org/show_bug.cgi?id=447558

--- Comment #2 from Fushan Wen  ---
The problem is in `m_screenMapper->removeScreen(m_screen, oldUrl);`. It does
not consider different activities.

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 447558] "Files linked to the current activity" makes desktops of other activities show nothing before restarting plasmashell

2021-12-27 Thread Paul Worrall
https://bugs.kde.org/show_bug.cgi?id=447558

Paul Worrall  changed:

   What|Removed |Added

 Ever confirmed|0   |1
 Status|REPORTED|CONFIRMED
 CC||p.r.worr...@gmail.com

--- Comment #1 from Paul Worrall  ---
Reproduced on 5.23.  

I needed to add step 2½:  Select activity A and use Dolphin to link a file to
the current activity (the file appears on the desktop of activity A as
expected).  

Operating System: KDE neon 5.23
KDE Plasma Version: 5.23.4
KDE Frameworks Version: 5.89.0
Qt Version: 5.15.3
Kernel Version: 5.11.0-43-generic (64-bit)
Graphics Platform: X11
Processors: 2 × AMD A6-6400K APU with Radeon(tm) HD Graphics
Memory: 7.7 GiB of RAM
Graphics Processor: AMD CEDAR

-- 
You are receiving this mail because:
You are watching all bug changes.