Bug #51364

Output of "Deprecated RTE properties in Page TSconfig" upgrade wizard is ambiguous

Added by Simon Schmidt about 8 years ago. Updated almost 4 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
RTE (rtehtmlarea + ckeditor)
Target version:
Start date:
2013-08-26
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

If the update exits with

"Update failed!
<p>Only page records were searched for deprecated properties. However, such properties can also be used in BE group and BE user records (prepended with page.). These are not searched nor updated by this wizard.</p>
<p>Page TSconfig may also be included from external files. These were not updated by this wizard. If required, the update will need to be done manually.</p>"

it should be no fail, but only a warning.

The HTML-Tags can also be removed.


Files


Related issues

Related to TYPO3 Core - Bug #53710: Improve error output for deprecated RTE properties during updateClosedNicole Cordes2013-11-17

Actions
#1

Updated by Stanislas Rolland over 7 years ago

  • Category set to RTE (rtehtmlarea + ckeditor)
#2

Updated by Stanislas Rolland over 7 years ago

  • Subject changed from Update Failed in "Deprecated RTE properties in Page TSconfig" should be warning not fail to Further improve output of "Deprecated RTE properties in Page TSconfig" upgrade wizard
  • Target version set to 6.2.0
#3

Updated by Stanislas Rolland over 7 years ago

Simon Schmidt wrote:

If the update exits with

"Update failed!
<p>Only page records were searched for deprecated properties. However, such properties can also be used in BE group and BE user records (prepended with page.). These are not searched nor updated by this wizard.</p>
<p>Page TSconfig may also be included from external files. These were not updated by this wizard. If required, the update will need to be done manually.</p>"

it should be no fail, but only a warning.

Either some deprecated properties were found, but the wizard was unable to replace them all automatically, in which case the wizard should report a failure, or no deprecated properties were found or there are no more deprecated properties left after the update, in which cases the wizard should report a success.

I agree that the warning is not needed here, as it is displayed when the wizard is first presented.

#4

Updated by Stanislas Rolland over 7 years ago

  • Tracker changed from Feature to Bug
  • Subject changed from Further improve output of "Deprecated RTE properties in Page TSconfig" upgrade wizard to Output of "Deprecated RTE properties in Page TSconfig" upgrade wizard is ambiguous
#5

Updated by Gerrit Code Review over 7 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 https://review.typo3.org/27369

#6

Updated by Stanislas Rolland over 7 years ago

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

Updated by Riccardo De Contardi almost 4 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF