Bug #65809: Autocompletion fails in TypoScript-Editor - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #65809

closed

Autocompletion fails in TypoScript-Editor

Added by Philipp Müller over 9 years ago. Updated over 7 years ago.

Status:

Closed

Priority:

Must have

Assignee:

Category:

t3editor

Target version:

Start date:

2015-03-18

Due date:

% Done:

Estimated time:

TYPO3 Version:

6.2

PHP Version:

Tags:

Complexity:

Is Regression:

Yes

Sprint Focus:

Stabilization Sprint

Description

Because of the bugfix #25594 the autocompletion of typoscript-code fails, when I try to create new Objects.

For example, when I create a new TEXT-Object like this:

lib.field_test = TEXT
lib.field_test {
  value = This is a test
}

The section after the first "lib." will be interpreted wrong:

Before the bugfix #25594 this worked correctly (see Screenshot 2) and when I remove the line 2603:

                    ch == "." ||

in typo3/sysext/t3editor/res/jslib/parse_typoscript/tokenizetyposcript.js, the problem is solved.

Files

Download all files

Screenshot.png (16.6 KB) Screenshot.png		Xavier Perseguers, 2015-04-13 11:38
screenshot1.png (8.4 KB) screenshot1.png	Wrong	Philipp Müller, 2015-04-22 11:43
screenshot2.png (8.31 KB) screenshot2.png	Okay	Philipp Müller, 2015-04-22 11:43

Related issues 4 (0 open — 4 closed)

Actions

Copy link

Updated by Suganja von Thaden over 9 years ago

Same bug! Anybody resolved this?

Actions

Copy link

Updated by Markus Klein over 9 years ago

Assignee set to Markus Klein

Actions

Copy link

Updated by Markus Klein over 9 years ago

Status changed from New to Accepted
Assignee changed from Markus Klein to Benni Mack

Actions

Copy link

Updated by Xavier Perseguers over 9 years ago

Target version changed from next-patchlevel to 6.2.12
Sprint Focus set to Stabilization Sprint

Actions

Copy link

Updated by Xavier Perseguers over 9 years ago

Description updated (diff)

Actions

Copy link

Updated by Xavier Perseguers over 9 years ago

File Screenshot.png Screenshot.png added
Description updated (diff)

Actions

Copy link

Updated by Xavier Perseguers over 9 years ago

File deleted (~~Screenshot 1.png~~)

Actions

Copy link

Updated by Xavier Perseguers over 9 years ago

File deleted (~~Screenshot 2.png~~)

Actions

Copy link

Updated by Xavier Perseguers over 9 years ago

Status changed from Accepted to Needs Feedback

Hi Philipp, I wrongly removed screenshots thinking there were same. As such I remember one did not have proper indentation but don't understand how to reproduce. Could you please either describe again the problem with screenshots (make sure NOT to include spaces within filename since one cannot embed them using !filename.png! syntax) or using just pre tags.

Is problem in 6.2 solely or in master as well?

Actions

Copy link

#10

Updated by Xavier Perseguers over 9 years ago

Target version changed from 6.2.12 to next-patchlevel
Sprint Focus deleted (~~Stabilization Sprint~~)

not yet ready, postponed

Actions

Copy link Download all files

#11

Updated by Philipp Müller over 9 years ago

File screenshot1.png screenshot1.png added
File screenshot2.png screenshot2.png added

Hi Xavier

Thank you for your response. The problem still exist in TYPO3 6.2.12.

You can reproduce it like this:

Create new a file, for example "fileadmin/test.ts"
Open it with t3editor*
Write the test-code (do not copy&paste it from here)

// Test-Code
lib.field_test = TEXT
lib.field_test {
  value = This is a test
}

Wrong:
Wrong

Okay:
Okay

PS: *You must add ".ts" in textfile_ext in TYPO3_CONF_VARS to edit ts-files:

$TYPO3_CONF_VARS['SYS']['textfile_ext'] = 'ts,txt,html,htm,css,tmpl,js,sql,xml,csv,php,php3,php4,php5,php6,phpsh,inc,phtml';

Actions

Copy link

#12

Updated by Bernd Wilke over 9 years ago

you can test it also with a simple typoscript-template record, editing the setup:

lib.mytest = COA
lib.mytest {
    // indention not recognized
    10 = TEXT
    10 {
        // indention recognized
       value = dummy
    }
    20 = TEXT
    20.value = dummy2
    20.stdWrap {
        // indention not recognized
        wrap = (|)
    }
}

marking all and pressing the tab key will result in all lines starting at the beginning (no indention) except the two lines inside 10

Actions

Copy link

#13