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

cordova config delete calling configstore incorrectly #447

Closed
3 tasks done
Noglen opened this issue Jun 30, 2019 · 0 comments · Fixed by #643
Closed
3 tasks done

cordova config delete calling configstore incorrectly #447

Noglen opened this issue Jun 30, 2019 · 0 comments · Fixed by #643
Milestone

Comments

@Noglen
Copy link

Noglen commented Jun 30, 2019

Bug Report

Problem

What is expected to happen?

Running cordova config delete removes the key from the config file

What does actually happen?

[mypath]\npm\node_modules\cordova\src\cli.js:139
conf.del(inputArgs[4]);
^

TypeError: conf.del is not a function
at module.exports ([mypath]\npm\node_modules\cordova\src\cli.js:139:18)
at Object. ([mypath]\npm\node_modules\cordova\bin\cordova:27:1)
at Module._compile (internal/modules/cjs/loader.js:701:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:712:10)
at Module.load (internal/modules/cjs/loader.js:600:32)
at tryModuleLoad (internal/modules/cjs/loader.js:539:12)
at Function.Module._load (internal/modules/cjs/loader.js:531:3)
at Function.Module.runMain (internal/modules/cjs/loader.js:754:12)
at startup (internal/bootstrap/node.js:283:19)
at bootstrapNodeJSCore (internal/bootstrap/node.js:622:3)

Information

cli.js is trying to call conf.del, but the correct method is actually conf.delete

Command or Code

cordova config delete

Environment, Platform, Device

Version information

9.0.0 ([email protected])

Checklist

  • I searched for existing GitHub issues
  • I updated all Cordova tooling to most recent version
  • I included all the necessary information above
@dpogue dpogue added this to the 13.0.0 milestone Jun 18, 2024
dpogue added a commit to dpogue/cordova-cli that referenced this issue Jun 18, 2024
dpogue added a commit to dpogue/cordova-cli that referenced this issue Jun 18, 2024
@dpogue dpogue closed this as completed in b57fb45 Jun 18, 2024
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 a pull request may close this issue.

2 participants