Bug #9090

Chaining of ->assign() does not work anymore

Added by Michael Feinbier almost 12 years ago. Updated over 11 years ago.

Status:
Resolved
Priority:
Must have
Category:
View
Target version:
-
Start date:
2010-07-30
Due date:
% Done:

100%

Estimated time:
Has patch:

Description

Since the latest release of Fluid the chaining of the ->assign() method does not work anymore

$this->view
  ->assign('request', $request)
  ->assign('languages', $this->languageRepository->findActive());

Throws error Fatal error: Call to a member function assign() on a non-object


Files

9090_FLOW3_v1.patch (18.7 KB) 9090_FLOW3_v1.patch patch for FLOW3 package Bastian Waidelich, 2010-08-02 15:24
9090_Fluid_v1.patch (9.97 KB) 9090_Fluid_v1.patch patch for Fluid package Bastian Waidelich, 2010-08-02 15:24
9090_tx_extbase_v1.patch (3.15 KB) 9090_tx_extbase_v1.patch patch for extbase extension Bastian Waidelich, 2010-08-02 15:24
9090_tx_fluid_v1.patch (10 KB) 9090_tx_fluid_v1.patch patch for fluid extension Bastian Waidelich, 2010-08-02 15:24

Related issues

Related to TYPO3.Flow - Task #9136: Clean up View implementationsResolvedBastian Waidelich2010-08-04

Actions
#1

Updated by Bastian Waidelich almost 12 years ago

  • Category changed from Core to View

The feature of chaining variable assignments was added with r2899
Since r4659 This feature was (unintentionally?) removed

Note: I think the assign functions should be moved back to FLOW3/Extbase AbstractView.
Or at least they should be defined in the global ViewInterface.

Note2: For v4 we should consider re-adding this for the Hotfix releases (1.1.x)

#2

Updated by Bastian Waidelich almost 12 years ago

Attached patches fixes the issue by adapting the interface and concrete view implementations. Additionally they add some unit tests and cleanups.

Sebastian Kurfuerst: Could you please check & apply (a subset of) the attached extbase & fluid patches for the Hotfix release? Afterwards feel free to re-assign this to me, so I can commit the rest to trunk.

#3

Updated by Bastian Waidelich almost 12 years ago

  • Assignee changed from Sebastian Kurfuerst to Bastian Waidelich

I'll take care of this (for trunks of FLOW3, Fluid, tx_extbase & tx_fluid)

#4

Updated by Bastian Waidelich almost 12 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100

Applied in changeset r4931.

Also available in: Atom PDF