Bug #85068
closedIpAnonymizationAdditionalFieldProvider causes javscript problem in Scheduler
100%
Description
Hi,
Please take a look at the attached gifs first
in the scheduler is a Javascript snippet which write the defaultNumberOfDays in the input Field "tx_scheduler[scheduler_tableGarbageCollection_numberOfDays]"
(see should-be-8.7.13.gif)
to get this information TableGarbageCollectionAdditionalFieldProvider set a Javascript var
$fieldHtml[] = '<script type="text/javascript">/*<![CDATA[*/<!--';
$fieldHtml[] = 'var defaultNumberOfDays = ' . json_encode($this->defaultNumberOfDays) . ';';
$fieldHtml[] = '// -->/*]]>*/</script>';
but IpAnonymizationAdditionalFieldProvider also sets this variable, but empty
// Add js array for default 'number of days' values
$fieldHtml[] = '<script type="text/javascript">/*<![CDATA[*/<!--';
$fieldHtml[] = 'var defaultNumberOfDays = ' . json_encode($this->defaultNumberOfDays) . ';';
$fieldHtml[] = '// -->/*]]>*/</script>';
the result in html is
<script type="text/javascript">/*<![CDATA[*/<!--
var defaultNumberOfDays = {"sys_log":180,"sys_history":30,"index_stat_search":90,"tx_formhandler_log":180};
// -->/*]]>*/</script></div></div></div>
...
...
...
<script type="text/javascript">/*<![CDATA[*/<!--
var defaultNumberOfDays = [];
// -->/*]]>*/</script></div></div></div>
Files
Updated by Riccardo De Contardi over 6 years ago
- Category set to Backend JavaScript
Updated by Frank Nägler over 6 years ago
- Status changed from New to Accepted
- Target version set to 9.3
Updated by Frank Nägler over 6 years ago
- Status changed from Accepted to In Progress
- Assignee set to Frank Nägler
Updated by Gerrit Code Review over 6 years ago
- Status changed from In Progress to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57136
Updated by Gerrit Code Review over 6 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57136
Updated by Gerrit Code Review over 6 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57136
Updated by Gerrit Code Review over 6 years ago
Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57164
Updated by Frank Nägler over 6 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 8ae18981104b486d1b09837a241c7083c7544b8c.
Updated by Gerrit Code Review over 6 years ago
- Status changed from Resolved to Under Review
Patch set 1 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57165
Updated by Gerrit Code Review over 6 years ago
Patch set 2 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57165
Updated by Frank Nägler over 6 years ago
- Status changed from Under Review to Resolved
Applied in changeset 3c0c210cb77493b98c07d8b2d1892600df1c7e8f.