Feature #20767

wrong getData behaviour with field parameter

Added by Myroslav Holyak almost 11 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Should have
Category:
-
Target version:
Start date:
2009-07-17
Due date:
% Done:

100%

PHP Version:
4.3
Tags:
Complexity:
Sprint Focus:
Remote Sprint

Description

Why we return only $this->data[$field] while GPVar and TSFE can have nested arrays and objects? I propose to correct that without any bad results.

(issue imported from #M11554)

tslib_content_getDdata_patch.txt View (504 Bytes) Administrator Admin, 2009-07-17 16:44

Associated revisions

Revision 469163eb (diff)
Added by Mathias Schreiber over 5 years ago

[FEATURE] Allow nested array access in getData type field

In order to make Typoscript more consistent we now allow to use nested
access to arrays for the datatype "field".

Resolves: #20767
Releases: master
Change-Id: I801054175c94151b7799300c575e34a1a29c7086
Reviewed-on: http://review.typo3.org/36040
Reviewed-by: Mathias Schreiber <>
Tested-by: Mathias Schreiber <>
Reviewed-by: Philipp Gampe <>
Tested-by: Philipp Gampe <>

History

#1 Updated by Oliver Hader almost 11 years ago

Can you please give an example how you use it?
How do you get nested database fields?

#2 Updated by Myroslav Holyak almost 11 years ago

I said nothing about only database fields. I can create my local cObject, ->start it with custom data array (e.g. with subarrays in fields) and make ->cObjGetSingle. So I can access to nested subarrays with ->getGlobal.

#3 Updated by Alexander Opitz about 7 years ago

  • Status changed from New to Needs Feedback
  • Target version deleted (0)

As this report is very old, is the handling in newer TYPO3 CMS Versions (like 6.0/6.1) more like you expect it?

#4 Updated by Myroslav Holyak about 7 years ago

Nope, still the same. Of course it is not actual for me now, but in general i think this could be a nice feature if using cObj from php code.

I think I could finally start using git, make a patch and make a code review.

#5 Updated by Alexander Opitz about 7 years ago

  • Status changed from Needs Feedback to New

#6 Updated by Mathias Schreiber over 5 years ago

  • Assignee set to Mathias Schreiber
  • Target version set to 7.2 (Frontend)
  • Sprint Focus set to Remote Sprint

#7 Updated by Gerrit Code Review over 5 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36040

#8 Updated by Gerrit Code Review over 5 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36040

#9 Updated by Gerrit Code Review over 5 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36040

#10 Updated by Gerrit Code Review over 5 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36040

#11 Updated by Mathias Schreiber over 5 years ago

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

#12 Updated by Riccardo De Contardi over 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF