Project

General

Profile

Actions

Feature #32169

closed

More useful preProcessValue hook

Added by Mario Matzulla over 12 years ago. Updated about 4 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Backend API
Start date:
2011-11-29
Due date:
% Done:

100%

Estimated time:
PHP Version:
Tags:
Complexity:
Sprint Focus:
Remote Sprint

Description

Hi,

please provide more information in the configuration array passed on ($theColConf). At least the table, col and the initial db value is needed in order to perform a useful transformation.

Thanks,
Mario


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Epic #88027: Properly handle Hooks/Signals and EventsClosedBenni Mack2022-04-22

Actions
Actions #1

Updated by Steffen Ritter about 12 years ago

  • Tracker changed from Bug to Feature
  • Target version deleted (4.7.0)
Actions #2

Updated by Oliver Hader over 11 years ago

  • Status changed from New to Accepted

Agreed... the preProcess hook is in my eyes non-sense, it just delivers the field configuration and there's even no possibility to really pre-process the value.

Actions #3

Updated by Oliver Hader over 11 years ago

  • Tracker changed from Feature to Bug
Actions #4

Updated by Oliver Hader over 11 years ago

  • Assignee changed from Jeff Segars to Oliver Hader
  • Target version set to 4.5.22
  • TYPO3 Version changed from 4.6 to 4.5
  • PHP Version set to 5.2
Actions #5

Updated by Oliver Hader over 11 years ago

  • Subject changed from preProcessValue and postProcessValue hook in t3lib_BEfunc need to deliver more information to Sloppy preProcessValue and postProcessValue hooks
Actions #6

Updated by Gerrit Code Review over 11 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/16405

Actions #7

Updated by Gerrit Code Review over 11 years ago

Patch set 1 for branch TYPO3_4-5 has been pushed to the review server.
It is available at http://review.typo3.org/16406

Actions #8

Updated by Ernesto Baschny almost 11 years ago

  • Status changed from Under Review to Rejected

In preProcessValue you can manipulate the TCA (which will then be applied in the next step). If you are still not happy with the result, you use postProcessValue (which gets the $value and $theColConf) to manipulate the value itself afterwads. So no change is required.

Actions #9

Updated by Ernesto Baschny almost 11 years ago

  • Tracker changed from Bug to Feature
  • Subject changed from Sloppy preProcessValue and postProcessValue hooks to More useful preProcessValue hook
  • Status changed from Rejected to Accepted
  • Assignee deleted (Oliver Hader)
  • Target version deleted (4.5.22)
  • PHP Version deleted (5.2)

The preProcessValue is indeed useless as it doesn't get the name of the column that is being handled, only "some TCA snippet" (impossible to guess for sure which field it is).

We won't add more hook APIs to 4.5, but we could add a new more useful hook for 6.2 (current master). So please focus this issue on that (new hook), and deprecate the the "old hook" - who knows who might have used it??? - on the way.

So I've changed this to a Feature request and adapted the subject to it. Open new review requests as soon you have a new suggestion.

Thanks for understanding!

Actions #10

Updated by Mathias Schreiber over 9 years ago

  • Assignee set to Mathias Schreiber
  • Target version set to 7.1 (Cleanup)
  • Sprint Focus set to Remote Sprint
Actions #11

Updated by Benni Mack almost 9 years ago

  • Target version changed from 7.1 (Cleanup) to 7.4 (Backend)
Actions #12

Updated by Susanne Moog over 8 years ago

  • Target version changed from 7.4 (Backend) to 7.5
Actions #13

Updated by Benni Mack over 8 years ago

  • Target version changed from 7.5 to 8 LTS
Actions #14

Updated by Riccardo De Contardi about 7 years ago

  • Target version changed from 8 LTS to 9.0
Actions #15

Updated by Susanne Moog about 6 years ago

  • Target version changed from 9.0 to Candidate for Major Version
Actions #16

Updated by Benni Mack about 5 years ago

  • Related to Epic #88027: Properly handle Hooks/Signals and Events added
Actions #17

Updated by Gerrit Code Review about 4 years ago

  • Status changed from Accepted 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 https://review.typo3.org/c/Packages/TYPO3.CMS/+/63877

Actions #18

Updated by Mathias Schreiber about 4 years ago

  • Assignee deleted (Mathias Schreiber)
Actions #19

Updated by Benni Mack about 4 years ago

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

Updated by Benni Mack about 4 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF