Skip to content
This repository was archived by the owner on Jan 6, 2023. It is now read-only.

Fix Plugin Parsing #36

Closed
wants to merge 1 commit into from
Closed

Fix Plugin Parsing #36

wants to merge 1 commit into from

Conversation

kentongray
Copy link
Contributor

Fix bug where having a plugin with preferences followed by a plugin with no preferences would not properly reset hasVariables and variable list.

Fix bug where having a plugin with  preferences followed by  a plugin with no preferences would not properly reset hasVariables and variable list
@kentongray
Copy link
Contributor Author

to me this looks like a bug introduced by pr #33 8884f4a#diff-2251310b0ef732442230e91e2715d586

@kentongray
Copy link
Contributor Author

@dwieeb I'm sure you get notified on this but just want to ping you so you know there is an issue on the latest release (0.6.2). I have a feeling this will be a common problem for end users with more than a couple plugins.

@kentongray
Copy link
Contributor Author

One more comment for SEO purposes, you will see an error like below if you are affected by this:

There was an error saving your current Ionic setup TypeError: Cannot read property 'URL_SCHEME' of null

(replace URL_SCHEME with completely random var of previous plugin)

@imhoffd
Copy link
Contributor

imhoffd commented Oct 20, 2015

Nice work. I'll have a new version out as soon as I can.

@imhoffd imhoffd closed this Oct 20, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants