Project

General

Profile

Actions

Bug #65824

closed

Changes in the constants editor effective after two-time save (7.2.0-dev)

Added by Harald no-lastname-given about 9 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Must have
Assignee:
Category:
Backend API
Target version:
Start date:
2015-03-18
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
7
PHP Version:
5.6
Tags:
Complexity:
no-brainer
Is Regression:
Yes
Sprint Focus:
Remote Sprint

Description

If changes in the constants editor are made, they will be effective after two-time save?!

I refer to the TYPO3 7.2.0-dev

I am using the latest Chrome as a browser

Thank you

Harald


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Task #64133: Cleanup sysext:tstemplateClosedMarkus Klein2015-01-062015-01-06

Actions
Actions #1

Updated by Sebastian Michaelsen almost 9 years ago

Confirmed. Actually the correct value is stored on the first save already, but when the form is reloaded it seems the values are read from the db before processing the updated values.

Actions #2

Updated by Sebastian Michaelsen almost 9 years ago

Seems like the bug was introduced with this commit: https://github.com/TYPO3/TYPO3.CMS/commit/94843c0d754e239b8f40b91d328ba30f769cb689#diff-c8fe1a2cbb0697db44406f6dad11cfc7

The change removed the global declaration for $theConstants and that's why the changes are not reflected in the form values.

Actions #3

Updated by Gerrit Code Review almost 9 years ago

  • Status changed from New 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 http://review.typo3.org/39575

Actions #4

Updated by Gerrit Code Review almost 9 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/39575

Actions #5

Updated by Markus Klein almost 9 years ago

  • Assignee set to Markus Klein
  • Priority changed from Should have to Must have
  • Target version set to 7.3 (Packages)
  • Complexity set to no-brainer
  • Is Regression changed from No to Yes
  • Sprint Focus set to Remote Sprint
Actions #6

Updated by Anonymous almost 9 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #7

Updated by Riccardo De Contardi over 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF