Project

General

Profile

Actions

Bug #64580

closed

ObjectAccess::getGettablePropertyNames name is misleading

Added by Cedric Ziel about 9 years ago. Updated about 7 years ago.

Status:
Closed
Priority:
Could have
Assignee:
-
Category:
Extbase
Target version:
-
Start date:
2015-01-29
Due date:
% Done:

0%

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

Description

This convenience method should just return the names of existing properties.

What it does is just returning a bunch of shortened getter names which is very inconvenient.

Actions #1

Updated by Christian Kuhn about 9 years ago

Please take a look if TYPO3.Flow changed this, the class is in there, too.
I'm working in this area too and have a WIP to synchronize getProperty() since this method has a cache implemented in flow that could be useful for extbase as well.

Actions #2

Updated by Cedric Ziel about 9 years ago

Same behaviour in Flow. (https://git.typo3.org/Packages/TYPO3.Flow.git/blob/HEAD:/Classes/TYPO3/Flow/Reflection/ObjectAccess.php#l265)

Honestly, for me it doesnt matter if it's in Flow or not. - The method name and its description is plain wrong.

Actions #3

Updated by Daniel Goerz about 8 years ago

  • Priority changed from Must have to Could have
Actions #4

Updated by Helmut Hummel over 7 years ago

  • Status changed from New to Needs Feedback

The method name and its description is plain wrong

The description says: "Returns an array of properties which can be get with the getProperty() method"
The scope of this method is this limited for further usage in Fluid or the mentioned method, but it has it use case.

What is your suggestion now? Change the description to be even more clear? Add new API for your use case? x?

Actions #5

Updated by Riccardo De Contardi about 7 years ago

  • Status changed from Needs Feedback to Closed

No feedback since 90 days > closing.

If you think that this is the wrong decision or that there is still work to be done on this topic, then please reopen this issue or open a new issue with a reference to this one.

Thank you.

Actions

Also available in: Atom PDF