The jquery.js file doesn't yet have a CVS $Id$ tag. Patch attached.

CommentFileSizeAuthor
jquery_cvs_id.patch17.21 KBUwe Hermann

Comments

drumm’s picture

drumm
he/him
Location NY, US
commented

I'm asking Steven about this since this is an unmodified file from the jQuery project.

gopherspidey’s picture

gopherspidey commented

I think there needs to be some location that you can find the jquery version number.

If that is not in the jquery file itself, there needs to be a location in the admin pages where you can find it.

gopherspidey’s picture

gopherspidey commented

Here is my idea to at least help this issue.

http://drupal.org/node/82831

Uwe Hermann’s picture

Uwe Hermann commented

+1 for the patch at http://drupal.org/node/82831. I'd still add a CVS $Id$, if only for matters of consistency. Also, it might happen in the future that Drupal-specific customizations are made, so the $Id$ could become imporant to distinguish different versions.

dries’s picture

dries commented

It doesn't really hurt to have a CVS Id version number and it can prove useful in case a security issue comes up. I'm all for adding it.

drumm’s picture

drumm
he/him
Location NY, US
commented
Status: Reviewed & tested by the community » Fixed

Waited long enough for Steven's review.

Committed to HEAD.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)