Line 603 in date/date_popup/date_popup.module has the following code:
$form['date_popup_timepicker']['#prefix'] .= t('<p>The Date Popup module uses a jQuery timepicker module. There is no "official" jQuery UI timepicker, and not everyone likes the one that is included here. If you do not want to use the timepicker, you can turn it off below and users will get a regular textfield instead.</p>');
$form['date_popup_timepicker']['#prefix'] hasn't already been defined, so the .= concatenation throws an notice. The attached patch removes the concatenation operator.
| Comment | File | Size | Author |
|---|---|---|---|
| date-prefixnotice.patch | 1.23 KB | mark trapp |
Comments
Comment #1
lauggh commentedI have the same issue, but with line 631.
I have the latest recommended release for the Date module, but that doesn't seem to fix the issue.
Comment #2
nimbletoad commentedSame issue with line 631. Looking for a patch or a fix. Any luck?
This is probably the incorrect way to fix this but the patch calls the wrong lines. I looked at the patch and noticed it is deleting a "." I went into the file and manually removed it and the error went away. If I knew how to fix the patch I would.
- $form['date_popup_timepicker']['#prefix'] .= t('
The Date Popup module uses a jQuery timepicker module. There is no "official" jQuery UI timepicker, and not everyone likes the one that is included here. If you do not want to use the timepicker, you can turn it off below and users will get a regular textfield instead.
');
+ $form['date_popup_timepicker']['#prefix'] = t('
The Date Popup module uses a jQuery timepicker module. There is no "official" jQuery UI timepicker, and not everyone likes the one that is included here. If you do not want to use the timepicker, you can turn it off below and users will get a regular textfield instead.
');
Comment #3
damienmckennaUnfortunately the D6 version of this module is no longer supported, but we appreciate the time you put into this. If this problem is relevant for D7 too, please reopen the issue. Thanks.