When using the Date Popup module I get this JS error and the rest of the page's JS doesn't load.
"uncaught exception: Syntax error, unrecognized expression: #"

Here is the generated code from "View Source":

"#edit-publish-on-timeEntry-popup-1":{"visible":{"#":{"checked":false}}},"#edit-unpublish-on-timeEntry-popup-1":{"visible":{"#":{"checked":false}}}}
CommentFileSizeAuthor
#6 date-popup-1243022-6.patch957 bytesfmosca

Comments

derekw’s picture

derekw commented

Update: must be a problem with the date popup module? Changing to text box resolves the problem.

jonathan1055’s picture

jonathan1055 commented
Status: Active » Postponed

I was thinking it was maybe not a scheduler problem. Just for info, what date popup version does it happen with?

I've seen lots of javascript errors in my firefox js log, but mostly they do not impact the operation so developers ignore them. Core has loads too ;-)

Jonathan

derekw’s picture

derekw commented

I'm using the current dev version of Date 7.x-2.x-dev.

mbopp’s picture

mbopp commented
Title: Bad javascript » Javascript error
Project: Scheduler » Date
Version: 7.x-1.x-dev » 7.x-2.x-dev
Status: Postponed » Active

I am experiencing this issue (unrecognized expression: #) in the js console when creating a date_popup form element. It only happens when TIME is in the date format. I am using 7.x-2.x-dev. It does NOT happen with 7.x-2.0-alpha3.

For reference as a test I did try removing the js in the date_popup module to see if that eliminated the error, and it did not.

fmosca’s picture

fmosca commented

subscribe

fmosca’s picture

fmosca commented
Status: Active » Needs review
StatusFileSize
new date-popup-1243022-6.patch957 bytes

Please review this patch: the problem seems to be that the element building code first checked for the existance of an "all day" checkbox, but later tried to use one anyway.

R.Muilwijk’s picture

R.Muilwijk commented

Had the same problem. The patch fixed it for me

karens’s picture

karens commented
Status: Needs review » Fixed

The same change needs to be made for the select element. Committed. Thanks!

adamdicarlo’s picture

adamdicarlo commented
Status: Fixed » Needs work

The commit for this issue introduced a syntax error:

http://drupalcode.org/project/date.git/blobdiff/e890776211006b75643ddbde...

See the very last + line of the diff. It looks like there's a missing if (.

karens’s picture

karens commented
Status: Needs work » Fixed

Ugh! Got it. Thanks.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

Anonymous’s picture

(not verified) commented
Issue summary: View changes

Clarification of the problem