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

Already on GitHub? Sign in to your account

@updateURL and @downloadURL are case-sensitive #1679

Closed
m4rcelofs opened this Issue Dec 12, 2012 · 5 comments

Comments

Projects
None yet
7 participants

The @updateurl and @downloadurl tags on the meta block are case-sensitive, meaning it's quite easy to make a mistake - in which case the 'Find Updates' will be disabled and there's no notification about what's wrong.

The fact it's case-sensitive is also not mentioned on the documentation.

Suggested accepted values are @updateurl and @downloadurl.

I think no need to change this name.
Because you should be careful whether it is case sensitive when coding.
Not let other people change for you.

Documentation at http://wiki.greasespot.net/Metadata_Block never mentions that the keywords are case sensitive, and all the other keywords are lower case, only those two with URL in them have uppercase letters.

I would rather see all keyword name parsing to be case insensitive. Metadata values should be case sensitive though.

Contributor

janekptacijarabaci commented Jul 14, 2013

However, if you do not change strings, maybe it could do something like:

https://github.com/greasemonkey/greasemonkey/blob/master/modules/parseScript.js#L57

-    var header = match[1];
+    var header = match[1].toLowerCase().replace(/url$/i, "URL");
Contributor

cletusc commented Jul 14, 2013

I am against having these case-insensitive. Element.innerHTML has "HTML" capitalized because it is an acronym. "URL" is an acronym too hence should be capitalized. All of the other keywords do not contain acronyms therefore should follow an all lowercase or at least camel-case formatting. The only odd metadata value I see is run-at which doesn't conform to either all lowercase or camel-case.

Contributor

sizzlemctwizzle commented Jul 28, 2013

Isn't JavaScript case-sensitive? Why would you think GM metablock values would be case-insensitive? This is a dupe of #1097. I'm -1 on this.

@arantius arantius modified the milestones: Pony, Bankruptcy Jul 25, 2017

@arantius arantius closed this Jul 25, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment