refactor(PluginInfo): cleanup & simplify (#132)

* refactor: PluginInfo

* _getTags: simplify

* _getTagsInPlatform: support a list of platforms

* getPodSpecs: remove useless undefined value filtering

* getPodSpecs: further improve formatting

* getPreferences: simplify

* getHookScripts: simplify

* getFrameworks: only determine vars once

* getFrameworks: improve readability

* ctor: improve

* ctor: better var name for root elem

* destructure xml-helpers

* inline addCordova helper

* apply lebab.destruct-params

* convert comments to JSDocs

* remove outdated TODO; sync is faster

* map tags explicitly

* make _getTags* private methods

Co-authored-by: エリス <erisu@users.noreply.github.com>

refactor(misc): cleanup & simplify (#131)

* refactor: PlatformJson

* refactor: CordovaCheck

* refactor(plist-helpers): rename underscore variable

* refactor(plist-helpers): rename pruneOBJECT function

* refactor(plist-helpers): formatting

* refactor(CordovaCheck): simplify isCordova w/ pwd

* refactor(plist-helpers): simplify array merge

* refactor(plist-helpers): simplify dict merge

* refactor(PlatformJson): use readJsonSync

Co-authored-by: エリス <erisu@users.noreply.github.com>

refactor(ConfigChanges): simplify (#130)

* refactor: ConfigChanges

* refactor: ConfigFile

* test(ConfigChanges): organize imports and constants

* refactor(ConfigChanges): reduce var scope

* refactor(ConfigChanges): remove unused statement

* refactor(ConfigChanges): simplify after removing useless statement

* refactor(ConfigChanges): drop useless if

* refactor(ConfigChanges): take method from variable

* refactor(ConfigFile): make use of Array#find

* refactor(ConfigFile): prefer cwd opt for glob

* refactor(ConfigFile): simplify return logic

* refactor(ConfigFile): use read-chunk in isBinaryPlist

* refactor: ConfigKeeper

* refactor(ConfigKeeper): use Map as cache

* refactor(ConfigKeeper): apply standard caching pattern

Co-authored-by: エリス <erisu@users.noreply.github.com>

refactor(xml-helpers): DRY & simplify (#128)

* refactor: xml-helpers

* DRY graftXML{Merge,Overwrite}

* Simplify graftXMLAttrs

* Simplify graftXML

* Simplify parseElementtreeSync

* Simplify removeDuplicatePreferences

* Better mergeXml

* Vastly simplify resolveParent

* Improve findInsertIdx formatting

* Simplify attribMatch

* Simplify equalNodes

* Simplify mergeChild

* Simplify pruneXMLRemove

* Simplify pruneXML

Co-authored-by: エリス <erisu@users.noreply.github.com>

fix: broken lock file from #95 (#129)

This regenerates `package-lock.json` from scratch as the file that was committed
in #95 did not match the dependencies from `package.json`.

refactor(munge-util): DRY & simplify (#127)

* remove unused process_munge export

* DRY {increment,decrement}_munge

* refactor: munge-util

* replace _ with modern JS

- replace underscore find methods

- use Object.assign instead of _.extend

  The fact that we are now only cloning own properties should be OK since
  `oldAttributes` should be a plain object

* improve variable names

* simplify mungeItems

* simplify deep_remove

* refactor process_munge to processArgs & getElements

We simplify getElements since it's always called with a top-level munge

* add JSDocs & type checking

Co-authored-by: エリス <erisu@users.noreply.github.com>

TEST: Test using GitHub workflows for CI (#95)

* Test using GitHub workflows for CI
* Commit package-lock for CI runs

test(ConfigFile): group & cleanup tests (#125)

* group tests in describe

* remove unused beforeEach

* reduce variable scope

* Make test titles unique

Co-Authored-By: エリス <erisu@users.noreply.github.com>

chore!: remove main export mungeUtil (#126)

Removes `mungeUtil` from the list of modules that are exported by
`cordova-common`. It was documented as _internal_ and we did not use it in any
of our repos.

refactor: FileUpdater (#123)

* add missing changes from "refactor: FileUpdater"

* remove unnecessary OR expression

* remove unnecessary parent dir creation

* DRY logging function default

* test: get rid of now-obsolete nullLogger arguments

* simplify building sourceMaps in mergeAndUpdateDir

* simplify OR over all update indicators

* use fast-glob for file globbing

Until now we used our own glob implementation based on minimatch.

The only difference to the old implementation is, that not every parent
directory of a matched file is included in the result. However, since do
not pass the dirMaps out of this module and any missing dirs will be
created by fs-extra, that should be an advantage, if anything.

* Simplify mergePathMaps

* DRY file copying code

* Handle removals early & flatten logic

* Always show verbose log

* DRY file copying code further

* Improve comments

* Handle special files early & simplify

* Flatten logic

* Normalize dirMap keys to pass tests on Windows

refactor: use ES6 classes where applicable (#119)

* chore(ConfigParser): move private helpers to bottom

* chore(PluginInfo): move ctor logic to top

* refactor(PluginInfo): drop useless intermediate vars

* refactor(PluginInfo): inline _parse* helpers

* refactor: use ES6 classes where applicable

* docs: remove obsolete JSDoc tags

refactor: use template strings where applicable (#118)

Applied `lebab --replace src --transform template` to the code and made some
minor adjustments.

refactor: use property shorthand notation (#116)

Applied `lebab --replace src --transform obj-shorthand` to the code.

refactor: transform var to let/const (#115)

Applied `lebab --replace src --transform let` to the code with minimal manual
changes afterwards.

refactor: do not alias `this` (#114)

After having switched to arrow functions everywhere in #113, this commit now
replaces all `this` aliases with lexical `this`.

refactor: use arrow functions where applicable (#113)

This commit makes use of arrow function expressions wherever applicable. Focus
is put on minimizing the diff instead of optimizing or formatting the code
touched by this commit.

refactor: move CordovaError module up (#120)

Moves the `CordovaError` module from `src/CordovaError/CordovaError.js` to `src/CordovaError.js`. The additional nesting makes no sense anymore since `src/CordovaError/CordovaExternalToolErrorContext.js` has been removed in #117.

refactor(CordovaError)!: remove unused features (#117)

* Remove CordovaExternalToolErrorContext

* Remove unused CordovaError.UNKNOWN_ERROR

* Remove unused verbose mode

* Remove unused getErrorCodeName

* Remove unused support for error codes

* Fix JavaDocs

chore: remove support for ubuntu platform (#111)

chore: consolidate eslint configs (#109)

style: drop jasmine env workaround (#108)

ESLint 6.7.0 has just been released and it contains the latest globals
for the jasmine environment. Thus we do not have to add them ourselves.

refactor: eslint setup (#105)

* replace dependencies with @cordova/eslint-config
* update eslint config
* eslint corrections
* chore: fix newline
* chore: re-add no-unused-vars after-used error
* chore: corrected no-unused-vars settings

test: always run code coverage during `npm test` (#107)

ci(travis): run codecov using npx in after_success (#106)

chore: bump production dependencies (#103)

chore: update jasmine dependencies & config (#101)

chore: replace instanbul w/ nyc (#100)

chore: drop unused & unneeded dependencies (#102)

* chore: drop promise-matchers dependency

* chore: drop osenv dependency

* chore: drop dependency which

chore: improve npm ignore list (#104)

Drop From Package
* all dot file and folders with .* (.github, .travis.yml, .eslintrc.yml, etc...)
  * .github/
  * .travis.yml
  * .eslintrc.yml
  * .eslintignore
  * .ratignore
* appveyor.yml

== PACKAGE REPORT ==
* BEFORE
npm notice package size:  41.5 kB
npm notice unpacked size: 179.4 kB
npm notice total files:   34

* AFTER
npm notice package size:  39.1 kB
npm notice unpacked size: 173.8 kB
npm notice total files:   24

feat: Replace addProperty with ES6 getters (#96)

* Replace addProperty with ES6 getters in main module

This allows static code analysis tools to better understand the module
dependencies. For example, you can use VS Code's code navigation feature
to jump to the module definition.

There's a slight difference in implementation, as the getters installed
by addProperty would only call `require` on first invocation and then
replace themselves with the actual module. But since require does
caching itself, I don't see how that would make a big difference in
terms of runtime.

* Replace addProperty with ES6 getters in ConfigFile

* Remove private module util/addProperty

fix: PluginInfoProvider for scoped plugins (#94)

* fix: PluginInfoProvider for scoped plugins
* Add test for PluginInfoProvider /w scoped plugin
* Normalize paths to use backslash on Windows

chore: update strip-bom@4 (#71)

chore: drop node 6 and 8 support (#90)

Use Array.prototype.find in CordovaError (#97)

Increment package version to 4.0.0-dev

Update version & RELEASENOTES.md for 3.2.1

Re-apply fix for failing CordovaError test (#93)

This fix was first introduced in #65 and reverted by #89

Fixes #49

Remove obsolete JSHint comments (#92)

Convert CordovaError to ES6 class (#87)

(windows) Add .jsproj as file extension for XML files (#62) (#78)

Fixes #62

revert: #24 CB-14108: fix incorrect count in config_munge (#89)

* Revert "CB-14108: fix incorrect count in config_munge in ios.json and android.json"

This reverts commit ce3801a184d34cc4c3ea08aeecd159227c6ae9e9.

* chore: eslint object-curly-spacing fix
* add CordovaError spec
* chore: fix eslint warnings for CordovaError spec
* fix CordovaError spec test
* chore: added additional AS project path check

Merge pull request #82 from brodybits/general-cleanup-with-eslint-update

General cleanup with eslint updates

eslint cleanup fixes for: padded-blocks

and remove padded-blocks exception from .eslintrc.yml

with src & spec fixed by the following command:

    npm run eslint -- --fix

(semi)standard eslint updates

- eslint-config-semistandard@14
- eslint-config-standard@13

with src & spec fixed by the following command:

    npm run eslint -- --fix

eslint cleanup fix for: no-prototype-builtins

as needed for the upcoming (semi)standard eslint package updates

Co-authored-by: Raphael von der Grün <raphinesse@gmail.com>
Co-authored-by: Christopher J. Brody <chris@brody.consulting>

eslint cleanup fixes: operator-linebreak (#86)

- remove operator-linebreak exception from .eslintrc.yml
- reformat ternary expresssions according to (semi)standard
- reformat error message expressions according to (semi)standard

as done by the following command, with no operator-linebreak
exception enabled:

    npm run eslint -- --fix

remove no-throw-literal lint exception not needed (#81)

Fix ESLint violations where applicable (#83)

Fixes most ESLint violations that were masked by ESLint comments. Two instances still remain. One is an acceptable usage of `eval` and is in `CordovaError` that should be fixed by its own commit.

Update to jasmine 3.4 & fix resulting spec failures (#80)

Promise handling cleanup in specs (#79)

* Proper expected rejection check in ActionStack spec

* Clean up Promise handling in PluginManager spec

* Remove usage of jasmine's done in sync specs

* Clean up Promise handling in superspawn spec

Do not ignore AppVeyor failures on Node.js 12 (#77)

Increment package version to 3.2.1-dev

Update version & RELEASENOTES.md for 3.2.0

Add Node.js 12 to CI Services (#68)

* Add Node.js 12 to CI Services

* TMP: Allow Node.js 12 Failure on AppVeyor

(ios) Plist document not indented with tabs

Xcode indents .plist documents with tabs, not spaces. Other parts of
Cordova already use tabs for indenting .plist documents and using spaces
directly conflicts with both cases.

Update fs-extra to v8 (#70)

Add example usage of podspec declarations

When trying to find more information about new `<podspec>` tag deprecating `<framework type="podspec" />` I found this example usage.
However it doesn't specify how to use declarations so I've copied example from [specs fixture](https://github.com/apache/cordova-common/blob/rel/3.1.0/spec/fixtures/plugins/org.test.plugins.withcocoapods/plugin.xml#L37-L47)

implement setPreference and setPlatformPreference

Catch leaked exceptions in superspawn and convert them to rejections

At least until Node.js 8, child_process.spawn will throw exceptions
instead of emitting error events in certain cases (like EACCES), Thus we
have to wrap the execution in try/catch to convert them into rejections.

Merge pull request #65 from raphinesse/test-fixes

Test fixes

Fix failing CordovaError test

Fixes #49

Properly handle promise in superspawn test

Add or update GitHub pull request and issue template

Set version to 3.1.1-dev

Update version & RELEASENOTES.md for release 3.1.0

Merge pull request #60 from brodybits/events-singleton

Cordova events singleton object

Update Cordova events to use a real singleton

Co-authored-by: Christopher J. Brody <chris.brody@gmail.com>
Co-authored-by: Darryl Pogue <darryl@dpogue.ca>

Refactor Cordova events into ES6 class

and replace var with const & let

Co-authored-by: Christopher J. Brody <chris.brody@gmail.com>
Co-authored-by: Darryl Pogue <darryl@dpogue.ca>

Merge pull request #59 from brodybits/events-spec-updates

Cordova events test updates in spec

Add Cordova events emit method tests to spec

based on existing test code

with other minor Cordova events test updates

Co-authored-by: Christopher J. Brody <chris.brody@gmail.com>
Co-authored-by: daserge <v-seshak@microsoft.com>

Explicit Cordova events section in spec

with forwardEventsTo method section indented

Refactor CordovaLogger to singleton class (#53)

Verbose test reporter (#55)

Co-authored-by: Raphael von der Grün <raphinesse@gmail.com>
Co-authored-by: Christopher J. Brody <chris.brody@gmail.com>

Cleanup indentation spacing in jasmine.json (#54)

Incremented package version to 3.0.1-dev

Updated version and RELEASENOTES.md for v3.0.0

Fix eslint object-curly-spacing complaints

Update fs-extra and eslint dependencies

Use `cross-spawn` for platform-independent spawning

Closes #50.

add method getPodSpecs and add related tests

CB-13496: Fix greedy regex in plist-helpers

CB-14108: fix incorrect count in config_munge in ios.json and android.json

* Fix to cordova prepare increment count at config_munge in platformJson
* Fix to cordova prepare when remove config-file case

Normalize whitespace (#46)

Fix xml-helpers issue by adding close tag

CB-13685 android: Update ConfigParser for Adaptive Icons

Make PlatformJson.generateMetadata more readable

type attributes for header-file is added

Simplify some helpers in util/xml-helpers (#40)

No functional changes. Just cleanup.

Pin Jasmine@~3.1.0 (#37)

CB-12016: (all) Removed cordova-registry-mapper dependency

CB-14099 osx: Fixed Resolve Config Path for OSX

 This closes #32

.travis.yml add spacing

Increment package version to 3.0.0-dev

Replace shelljs calls with fs-extra & which (#21)

added package-lock.json to .rateignore

CB-14112: Incremented package version to 2.3.0-dev

CB-14112: Updated version and RELEASENOTES.md for release 2.2.2

Use strip-bom to strip BOM

Simplify element removal in ConfigParser

Use .includes and .startsWith instead of .indexOf