Skip to content

Correct syntax error stopping update check dialog from opening when a previous update had been downloaded and there are incompatible add-ons.#15285

Merged
michaelDCurran merged 1 commit into
betafrom
i15284
Aug 14, 2023
Merged

Conversation

@michaelDCurran

@michaelDCurran michaelDCurran commented Aug 14, 2023

Copy link
Copy Markdown
Member

Link to issue number:

Fixes #15028
Fixes #15284

Summary of the issue:

the Check For Updates dialog fails to open if a previous update has already been downloaded (is pending) and there are incompatible add-ons.
This is due to a syntax error when concatinating a string.

x += + y

The second plus is invalid and not necessary.

Description of user facing changes

the Check For Updates dialog will no longer fail to open if a previous update has already been downloaded (is pending) and there are incompatible add-ons.

Description of development approach

Remove the invalid plus character.

Testing strategy:

  • Create try build
  • Install try build
  • Ensure one or more add-ons are installed that are incompatible with 2023.X.
  • Open python console, and set versionInfo.updateVersionType to 'beta', versionInfo.version to '2022.4', and then run updateCheck.initialize()
  • Download the offered update from the dialog that automatically appears, but dismiss it before actually installing it.
  • Restart NVDA.
  • Open python console, and set versionInfo.updateVersionType to 'beta', versionInfo.version to '2022.4', and then run updateCheck.initialize()
  • A dialog should automatically appear asking if you want to install the pending update already downloaded. The dialog will also mention that there are incompatible add-ons. This is the string that was failing.

Known issues with pull request:

None known.

Change log entries:

New features
Changes
Bug fixes
For Developers

Code Review Checklist:

  • Pull Request description:
    • description is up to date
    • change log entries
  • Testing:
    • Unit tests
    • System (end to end) tests
    • Manual testing
  • API is compatible with existing add-ons.
  • Documentation:
    • User Documentation
    • Developer / Technical Documentation
    • Context sensitive help for GUI changes
  • UX of all users considered:
    • Speech
    • Braille
    • Low Vision
    • Different web browsers
    • Localization in other languages / culture than English
  • Security precautions taken.

… previous update had been downloaded and there are incompatible add-ons.
@michaelDCurran michaelDCurran requested a review from a team as a code owner August 14, 2023 00:15
@michaelDCurran michaelDCurran requested review from seanbudd and removed request for a team August 14, 2023 00:15
@seanbudd seanbudd added this to the 2023.2 milestone Aug 14, 2023
@seanbudd

Copy link
Copy Markdown
Member

I discovered that this should also close: #15028

I'm not sure how that issue missed getting the milestone assigned, likely my error.

@michaelDCurran michaelDCurran merged commit 538caec into beta Aug 14, 2023
@michaelDCurran michaelDCurran deleted the i15284 branch August 14, 2023 01:30
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.

2 participants