Bug #3359

Restructure Class structure in Fluid Core

Added by Sebastian Kurfuerst about 6 years ago. Updated almost 5 years ago.

Status:Resolved Start date:2009-05-18
Priority:Should have Due date:
Assigned To:Sebastian Kurfuerst % Done:

100%
Category:-
Target version:1.0.0 alpha 1
Has patch: Affected Flow version:

Description

I propose to change the Fluid Core classes like that:

Core/
ViewHelper/
Facets/
AbstractViewHelper
TagBasedViewHelper
TagBuilder
ViewHelperContext
ViewHelperException?
ViewHelperInterface
ViewHelperArguments -> Arguments?
Parser/
SyntaxTree/
ParsedTemplateInterface
ParsingException?
ParsingState
TemplateParser

This changes the following public API classes:
Core\TemplateParser -> Core\Parser\TemplateParser.

Associated revisions

Revision 4fd18e87
Added by Sebastian Kurfuerst about 6 years ago

Fluid:
  • Restructuring node architecture (fixes #3359)

History

#1 Updated by Bastian Waidelich about 6 years ago

So you would have:

Core/
    ViewHelper/
        Facets/
            ChildNodeAccessFacet
            PostParseInterface
        AbstractViewHelper
        TagBasedViewHelper
        TagBuilder
        ViewHelperContext
        ViewHelperException?
        ViewHelperInterface
        ViewHelperArguments -> Arguments?
    Parser/
        SyntaxTree/
            AbstractNode
            ArrayNode
            ObjectAccessorNode
            RootNode
            TextNode
            ViewHelperNode
        ParsedTemplateInterface
        ParsingException?
        ParsingState
        TemplateParser
View/
    [...]
ViewHelpers/
    [...]
Exception
Fluid

correct?

Seems good to me!
I'd rename ViewHelper/ViewHelperException to ViewHelper/Exception, ViewHelper/ViewHelperArguments to ViewHelper/Arguments and Parser/ParsingException to Parser/Exception

#2 Updated by Sebastian Kurfuerst about 6 years ago

  • Category set to 444
  • Status changed from New to Resolved
  • % Done changed from 0 to 100

fixed :

Also available in: Atom PDF