Bug #35412

groupedFor cannot group by DateTime-Objects

Added by Stefan Froemken about 1 year ago. Updated 4 months ago.

Status:Resolved Start date:2012-03-30
Priority:Should have Due date:
Assignee:- % Done:

100%
Category:Fluid: ViewHelpers
Target version:Fluid 6.1
Complexity:
Votes: 0

Description

Hello MVC-List,

nothing more to say.

The Fluid-ViewHelper groupedFor cannot group by as DateTime-Object defined properties.

Stefan

As long as this is not solved I added a new getter in my model:

/** * @return integer
*/
public function getFristDatumAsInteger() {
return $this->fristDatum->format('U');
}

Associated revisions

Revision 936a40a2
Added by Kilian Hann 4 months ago

[TASK] Add test for GroupedForViewHelper to support DateTime Objects

The DateTime Object was fixed before, but the GroupedForViewHelperTest
still misses a test to demonstrate the possibility to group items
subsequently after a property defined as DateTime Object.

So the patch originally meant to apply is dropped, but we keep the test.

Change-Id: I658d99afd82e9572f070ac9daef25416031bc215
Fixes: #35412
Releases: 6.1
Reviewed-on: https://review.typo3.org/10440
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Markus Günther
Tested-by: Markus Günther
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring

History

#1
Updated by Gerrit Code Review about 1 year ago

  • Status changed from New to Under Review

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

#2
Updated by Markus Günther 11 months ago

  • Has patch changed from No to Yes

#3
Updated by Gerrit Code Review 8 months ago

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

#4
Updated by Anja Leichsenring 6 months ago

  • Project changed from Extbase MVC Framework to Fluid
  • Category changed from 963 to Fluid: ViewHelpers

#5
Updated by Anja Leichsenring 6 months ago

  • Target version changed from 4.5.15 to Fluid 6.1

#6
Updated by Gerrit Code Review 4 months ago

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

#7
Updated by Gerrit Code Review 4 months ago

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

#8
Updated by Gerrit Code Review 4 months ago

Patch set 5 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/10440

#9
Updated by Anonymous 4 months ago

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

Also available in: Atom PDF