Skip to content

When disabling and then immediately re-enabling an add-on or vice versa, the add-on status now correctly reverts to what it was previously.#6299

Merged
jcsteh merged 1 commit into
masterfrom
undoAddonDisable
Sep 16, 2016
Merged

When disabling and then immediately re-enabling an add-on or vice versa, the add-on status now correctly reverts to what it was previously.#6299
jcsteh merged 1 commit into
masterfrom
undoAddonDisable

Conversation

@jcsteh

@jcsteh jcsteh commented Aug 24, 2016

Copy link
Copy Markdown
Contributor

When an add-on is in the pending disable set and the user enables it, the user is just undoing the disable; i.e. there is nothing to do. Therefore, just remove it from the pending disable set; don't also add it to the pending enable set. Make a similar change for undoing an enable.

…sa, the add-on status now correctly reverts to what it was previously.

When an add-on is in the pending disable set and the user enables it, the user is just undoing the disable; i.e. there is nothing to do. Therefore, just remove it from the pending disable set; don't also add it to the pending enable set. Make a similar change for undoing an enable.
@jcsteh

jcsteh commented Aug 24, 2016

Copy link
Copy Markdown
Contributor Author

STR:

  1. Ensure there is at least one add-on installed.
  2. Open the Add-ons Manager.
  3. Select an add-on.
  4. Press Disable.
  5. Immediately press Enable.
    • Expected: The status should revert to "running".
    • Result before the fix: The status is "enable".

@jcsteh

jcsteh commented Aug 24, 2016

Copy link
Copy Markdown
Contributor Author

@feerrenrut, can you review please? Thanks!

CC @josephsl.

@josephsl

Copy link
Copy Markdown
Contributor

Hi, in regards to user guide, should the possibly misleading sentence be removed? Thanks.

From: James Teh [mailto:notifications@github.com]
Sent: Tuesday, August 23, 2016 8:51 PM
To: nvaccess/nvda nvda@noreply.github.com
Cc: Joseph Lee joseph.lee22590@gmail.com; Mention mention@noreply.github.com
Subject: Re: [nvaccess/nvda] When disabling and then immediately re-enabling an add-on or vice versa, the add-on status now correctly reverts to what it was previously. (#6299)

@feerrenrut https://github.com/feerrenrut , can you review please? Thanks!

CC @josephsl https://github.com/josephsl .


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub #6299 (comment) , or mute the thread https://github.com/notifications/unsubscribe-auth/AHgLkEMzWp-FoGATUDwbqKTH39tVVY71ks5qi7-VgaJpZM4JrnCD .

@jcsteh

jcsteh commented Aug 24, 2016 via email

Copy link
Copy Markdown
Contributor Author

@josephsl

Copy link
Copy Markdown
Contributor

Hi, confirmed the STR. Thanks.

From: James Teh [mailto:notifications@github.com]
Sent: Tuesday, August 23, 2016 9:12 PM
To: nvaccess/nvda nvda@noreply.github.com
Cc: Joseph Lee joseph.lee22590@gmail.com; Mention mention@noreply.github.com
Subject: Re: [nvaccess/nvda] When disabling and then immediately re-enabling an add-on or vice versa, the add-on status now correctly reverts to what it was previously. (#6299)

That's a separate issue I think. See the steps to reproduce earlier in
this issue for details on what this PR is fixing.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub #6299 (comment) , or mute the thread https://github.com/notifications/unsubscribe-auth/AHgLkDPTELE65FvfIJxGIrJ6f5O0fpVpks5qi8SSgaJpZM4JrnCD .

@feerrenrut

Copy link
Copy Markdown
Contributor

Yep, change looks good to me.

jcsteh added a commit that referenced this pull request Aug 29, 2016
@nvaccessAuto nvaccessAuto assigned jcsteh and unassigned feerrenrut Aug 29, 2016
@jcsteh jcsteh merged commit 05d74c7 into master Sep 16, 2016
@nvaccessAuto nvaccessAuto added this to the 2016.4 milestone Sep 16, 2016
@jcsteh jcsteh deleted the undoAddonDisable branch September 16, 2016 03:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants