Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

syncLocalSettings does not support replacing setting with "False" value #516

Closed
robertpeteuil opened this issue Mar 14, 2018 · 5 comments
Closed
Labels
Milestone

Comments

@robertpeteuil
Copy link

Visual Studio Code Version : 1.21.0 (1.21.0)
Code Settings Sync Version : 2.9.0
Operating System : macOS 10.12.7, Ubuntu 16.04
Occurs On: Download
Proxy Enabled: No
GIst Id: (private)

Problem:

  • If an entry in the replaceCodeSettings: section of syncLocalSettings.json sets the value of a setting to False, the setting is deleted from the local machines settings file (instead of setting it to a False value).
  • This is an issue as there are many setting which default to True and require explicit values of False to turn them off.

Scenario:

  • I'm syncing between multiple machines and only want the Docker sidebar visible on my Linux machines.
  • Hiding the sidebar required adding "docker.showExplorer": false to settings, so I set:
    • "docker.showExplorer": false to syncLocalSettings.json on my Macs
    • "docker.showExplorer": true to syncLocalSettings.json on my Linux machines.
  • When I sync, the setting "docker.showExplorer" is completely deleted from the settings file on my Macs.
  • Since it defaults to True, deleting it is the same as setting it to True.

It appears that replaceCodeSettings assumes a value of False means it should delete the setting. How can I explicitly set a setting as False on specific machines?

@leepowelldev
Copy link
Contributor

I am getting the same when trying to set http.proxyStrictSSL: false

@leepowelldev
Copy link
Contributor

PR here: #521

@robertpeteuil
Copy link
Author

@leepowellcouk Cool, thank you!

@shanalikhan
Copy link
Owner

Thanks for PR.
Will be released in next version.

@shanalikhan shanalikhan added this to the v2.9.1 milestone Mar 24, 2018
shanalikhan pushed a commit that referenced this issue Mar 24, 2018
shanalikhan added a commit that referenced this issue May 1, 2018
* Fix for #516

* Resolves #510 - support OSS build of vscode

* add support for xdg environment paths

* readme: fixed typos

* v291

* Package updates
@shanalikhan
Copy link
Owner

Released in v2.9.1
let me know if there is any issue.

shanalikhan pushed a commit that referenced this issue Aug 17, 2018
… enabled. (#604)

* Fix for #516

* Update to fix `ignoredExtensions` where extensions were deleted if `removeExtensions` was enabled.
shanalikhan added a commit that referenced this issue Aug 26, 2018
* german localization

* corrected naming of locale

* german localization

* corrected naming of locale

* fix: missing partial i18n translation

* fix: missing partial i18n translation

* fix: add missing key for German language file

* fix: typo

* Minor Wording Changes

The text on some parts looked strange and not correct, thought I'd update a few for grammar and looks.

* chore: update .gitignore

* chore: add tslint and prettier rules

* chore: add format script

* chore: update teslint rules to make it defined mutilple class in one file

* chore: run tslint before publish extension

* refactor: format the code with prettier

* chore: update tsconfig and tslint

* refactor: refactor utils.ts

* refactor: refactor localize.ts

* refactor: refactor setting.ts

* refactor: refactor environmentPath.ts

* refactor: refactor fileService.ts

* chore: update teslint rules

* refactor: replace github package with @octokit/rest

* chore: update teslint rules

* refactor: refactor commons.ts

* refactor: refactor pluginService.ts

* refactor: extension.ts

* feat: replace fs module with fs-extra, and use async await instead of callback style

* feat: update deps for vscode/typescript/adm-zip/tslint

* refactor: refactor callback style with async await

* remove unused dependencies ncp and rimraf, use fs-extra instead

* fix: invalid @octokit/rest constructor options

* chore: update travis script

* fix: lstat should be async no sync

* fix: user agent for proxy

* pref: improve performance when start up. use async instead of sync method #472

* fix: implement promisify and fix lockfile

* feat: use lockfile instead of proper-lockfile.

* pref: improve performance

* refactor: add sync class and clean up extension file. it should improve startup performance #472

* refactor: add lockfile.ts

* fix: typo ShowSummaryOutput

* refactor: refactor download function

* fix: lockfile

* refactor: rename 'en' to 'env'

* refactor: remove unused activationEvents

* refactor: refactor upload logic, make it clean

* style: rename lockfile function name, make it same style with other

* refactor: sync advance options

* chore: not allow unuser locals and params. make it clean

* refactor: remove unused local and params

* refactor: remove unneed await

* pref: improve localize performance, init when extension be actived

* fix: make sure resolve language merger with default language.

* Update tutorial message

* Fix slack img

* Update path to imgur link

* Update tutorial link again

* Extension Installation CLI Added
TSLint Improved
#434
#590
#513
#337

* Summary improved

* Github Api Code Improvements

Anonymous Gist Code Removed
Initializtion in contructor
github api updated
proxy addition
github enterprise support updated

* Ignored extensions can be accidentally deleted if removeExtensions is enabled. (#604)

* Fix for #516

* Update to fix `ignoredExtensions` where extensions were deleted if `removeExtensions` was enabled.

* #604

* Extension summary

#577

* Gist Name change
#513

* #611

* fix: error translation

* #611 - Changelog

CLI Improvements

* #611

* #611
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants