Bug #38375
Missing semicolon in jquery.ui.datepicker breaks JS concatenation
| Status: | Resolved | Start date: | 2012-06-25 | |
|---|---|---|---|---|
| Priority: | Could have | Due date: | ||
| Assignee: | Alex Kellner | % Done: | 100% |
|
| Category: | Frontend Plugin | |||
| Target version: | 2.0.5 | |||
| Votes: | 2 (View) |
Description
There is no semicolon at the end of Resources/Public/Js/jquery.ui.datepicker.min.js in powermail 2.0.0 which breaks typo3 js concatenation due to syntax errors.
History
Updated by Alex Kellner 11 months ago
- Status changed from New to Accepted
- Assignee changed from Alexander Grein to Alex Kellner
- Target version set to 2.0.2
Updated by Alex Kellner 9 months ago
- Status changed from Accepted to Rejected
- Assignee changed from Alex Kellner to Felix Buenemann
- % Done changed from 0 to 100
I did a little research and I don't think that this is a issue.
If your datepicker was not working, this could be related to the "marker wasn't generated bug" on #37959
Please check if the column "marker" of the datefield was filled in the database.
Updated by Wolfgang Wagner 7 months ago
I can confirm this behavior in Powermail 2.0.4.
If I set config.concatenateJs = 1, then the JavaScript-Validation does not work. But if I add a semicolon at the end of jquery.ui.datepicker.min.js, then the concatenation works as expected.
Updated by Alex Kellner 6 months ago
- Status changed from Rejected to Under Review
- Assignee changed from Felix Buenemann to Alex Kellner
- Priority changed from Must have to Could have
- Target version changed from 2.0.2 to 2.x
Thx for the report
Basicly I don't want to change anything of the given UI-Script. But if there is no better solution, we should do it.
Updated by Alex Kellner 6 months ago
- % Done changed from 100 to 30
Updated by Alex Kellner 6 months ago
- Status changed from Under Review to Accepted
- Target version changed from 2.x to 2.0.5
- % Done changed from 30 to 100
Fixed in current trunk - ready for TER upload 2.0.5
Updated by Alex Kellner 6 months ago
- Status changed from Accepted to Resolved
Published in Powermail version 2.0.5 - available in a couple of hours in TER