Bug #36236

Call to method not being backed by an interface in Http RequestHandler

Added by Adrian Föder over 3 years ago. Updated about 3 years ago.

Status:Resolved Start date:2012-04-17
Priority:Must have Due date:
Assigned To:Robert Lemke % Done:

100%

Category:Http
Target version:TYPO3 Flow Base Distribution - 1.1
PHP Version: Complexity:
Has patch:No Affected Flow version:Git master

Description

in Http's RequestHandler's handleRequest() method, there is

$this->router->setRoutesConfiguration($this->routesConfiguration);

in my case case of a Widget Request, $this->router is TYPO3\Fluid\Core\Widget\WidgetRouter which hasn't a setRoutesConfiguration() method.

So now there are few issues with that:

  • the $this->router is annotated to be \TYPO3\FLOW3\Mvc\Routing\Router, not ...RouterInterface
  • the RouterInterface, however, hasn't the setRoutesConfiguration() method.
  • the TYPO3\Fluid\Core\Widget\WidgetRouter doesn't implement from that interface
  • and, as said, the TYPO3\Fluid\Core\Widget\WidgetRouter also doesn't have the setRoutesConfiguration().

Related issues

related to TYPO3.Fluid - Bug #36500: Ajax widgets can't be executed Resolved 2012-04-24

History

#1 Updated by Robert Lemke over 3 years ago

  • Status changed from New to Accepted
  • Assigned To set to Robert Lemke

#2 Updated by Gerrit Code Review over 3 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/10731

#3 Updated by Gerrit Code Review over 3 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/10731

#4 Updated by Gerrit Code Review over 3 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/10734

#5 Updated by Gerrit Code Review about 3 years ago

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11101

#6 Updated by Gerrit Code Review about 3 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11101

#7 Updated by Gerrit Code Review about 3 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11101

#8 Updated by Gerrit Code Review about 3 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11101

#9 Updated by Karsten Dambekalns about 3 years ago

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

Also available in: Atom PDF