Project

General

Profile

Actions

Bug #77957

closed

Fluid ViewHelper namespace does not respect camelcase

Added by Markus Kugler about 8 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Fluid
Target version:
-
Start date:
2016-09-15
Due date:
% Done:

0%

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

Description

If you define your namespace like following:

{namespace nameSpace=Vendor\Ext\ViewHelpers}

The namespace will not be resolved using a case sensetive OS like Linux. This also will not work if your first character is upperCase.

I didn't find anything in the Documentation or Source that this should not be allowed. I don't know if this is also the case in earlier Versions.

Actions #1

Updated by Stefan Froemken about 8 years ago

  • Status changed from New to Accepted

You're right

$SPLIT_PATTERN_TEMPLATE_DYNAMICTAGS has changed in newer TYPO3 versions from

... (?:(?:NAMESPACE):[a-zA-Z0-9\\.]+) ...

to

... (?:(?:[a-z0-9\\.]*):[a-zA-Z0-9\\.]+) ...

but in $SCAN_PATTERN_TEMPLATE_VIEWHELPERTAG it is still

... (?P<NamespaceIdentifier>[a-zA-Z0-9\\.]*): ...

So, as you can see, for splitting the template up into pieces, Fluid VHs with uppercase letters will not be detected.

Stefan

Actions #2

Updated by Stefan Froemken about 8 years ago

I have created a pull request at Github: https://github.com/TYPO3Fluid/Fluid/pull/143

Actions #3

Updated by Claus Due about 8 years ago

  • Status changed from Accepted to Resolved

Fixed since updating Fluid dependency to 1.0.11 (currently at 1.1.1 awaiting 2.0.0).

Actions #4

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF