Project

General

Profile

Actions

Bug #66526

closed

t3editor have strange code duplication in JS file "res/jslib/parse_typoscript/tokenizetyposcript.js"

Added by Guillaume Crico about 9 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Should have
Category:
t3editor
Start date:
2015-04-21
Due date:
% Done:

100%

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

Description

It looks like the whole code had been duplicated.

And worst: there are also some differences between the two parts.

This strange code duplication is here since the initial t3editor import in the repository.


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Bug #72389: t3editor tokenizetyposcript.js has double contentClosedAlexander Opitz2015-12-22

Actions
Actions #1

Updated by Riccardo De Contardi almost 9 years ago

file: /sysext/t3editor/Resources/Public/Javascript/parse_typoscript/tokenizetyposcript.js

seems present also on 6.2.14 (file:/sysext/t3editor/res/jslib/parse_typoscript/tokenizetyposcript.js)

Actions #2

Updated by Riccardo De Contardi over 8 years ago

Still present in 7.6-dev (latest master)

Actions #3

Updated by Riccardo De Contardi over 8 years ago

  • Category set to t3editor

Still present in 7.6-dev (latest master)

Actions #4

Updated by Alexander Opitz over 8 years ago

  • Status changed from New to In Progress
  • Assignee set to Alexander Opitz
  • Target version set to Candidate for patchlevel
Actions #5

Updated by Gerrit Code Review over 8 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/45407

Actions #6

Updated by Gerrit Code Review over 8 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/45407

Actions #7

Updated by Gerrit Code Review over 8 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/45407

Actions #8

Updated by Gerrit Code Review over 8 years ago

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

Actions #9

Updated by Gerrit Code Review over 8 years ago

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

Actions #10

Updated by Gerrit Code Review over 8 years ago

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

Actions #11

Updated by Gerrit Code Review about 8 years ago

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

Actions #12

Updated by Alexander Opitz about 8 years ago

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

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF