Project

General

Profile

Actions

Bug #64580

closed

ObjectAccess::getGettablePropertyNames name is misleading

Added by Cedric Ziel almost 10 years ago. Updated almost 8 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 almost 10 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 almost 10 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 over 8 years ago

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

Updated by Helmut Hummel about 8 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 almost 8 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