<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.kde.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugs.kde.org/"
          
          maintainer="sysadmin@kde.org"
>

    <bug>
          <bug_id>457208</bug_id>
          
          <creation_ts>2022-07-27 15:19:20 +0000</creation_ts>
          <short_desc>Closing when updating for the second time creates second job notification</short_desc>
          <delta_ts>2022-08-08 12:23:46 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>2</classification_id>
          <classification>Applications</classification>
          <product>Discover</product>
          <component>discover</component>
          <version>master</version>
          <rep_platform>Other</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>NOR</priority>
          <bug_severity>minor</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Nicolas Fella">nicolas.fella</reporter>
          <assigned_to name="Plasma Bugs List">plasma-bugs-null</assigned_to>
          <cc>aleixpol</cc>
    
    <cc>bugseforuns</cc>
    
    <cc>nate</cc>
          
          <cf_commitlink>https://invent.kde.org/plasma/discover/commit/389725b6f57466fbdf9f73a00223f713ae5345c0</cf_commitlink>
          <cf_versionfixedin></cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>2141876</commentid>
    <comment_count>0</comment_count>
    <who name="Nicolas Fella">nicolas.fella</who>
    <bug_when>2022-07-27 15:19:20 +0000</bug_when>
    <thetext>STEPS TO REPRODUCE
1. Start an update
2. Close the window
3. Open window from SNI
4. Close again

OBSERVED RESULT
Two notifications about the update progress

EXPECTED RESULT
Only one notification about the update process


SOFTWARE/OS VERSIONS
KDE Plasma Version: master
KDE Frameworks Version: master
Qt Version: 5.15.5-kde</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2141940</commentid>
    <comment_count>1</comment_count>
    <who name="Nate Graham">nate</who>
    <bug_when>2022-07-27 20:04:22 +0000</bug_when>
    <thetext>Can reproduce.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2141987</commentid>
    <comment_count>2</comment_count>
    <who name="Bug Janitor Service">bug-janitor</who>
    <bug_when>2022-07-28 00:56:47 +0000</bug_when>
    <thetext>A possibly relevant merge request was started @ https://invent.kde.org/plasma/discover/-/merge_requests/340</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2142310</commentid>
    <comment_count>3</comment_count>
    <who name="Nate Graham">nate</who>
    <bug_when>2022-07-29 14:54:21 +0000</bug_when>
    <thetext>Git commit 389725b6f57466fbdf9f73a00223f713ae5345c0 by Nate Graham, on behalf of Aleix Pol.
Committed on 29/07/2022 at 14:52.
Pushed by ngraham into branch &apos;master&apos;.

Make sure the Job notification goes as we restore the view

Otherwise we get multiple of them.

M  +12   -3    discover/DiscoverObject.cpp
M  +1    -0    discover/DiscoverObject.h
M  +1    -1    libdiscover/backends/DummyBackend/DummyTransaction.cpp

https://invent.kde.org/plasma/discover/commit/389725b6f57466fbdf9f73a00223f713ae5345c0</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2144267</commentid>
    <comment_count>4</comment_count>
    <who name="Nicolas Fella">nicolas.fella</who>
    <bug_when>2022-08-08 12:23:46 +0000</bug_when>
    <thetext>*** Bug 457622 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>