Bug #25704
closed
Use consistent class names
100%
Description
Could we decide for either Tx or tx and not use both?
Tx_Workspaces_Controller_AbstractController
Tx_Workspaces_Controller_PreviewController
Tx_Workspaces_Controller_ReviewController
tx_Workspaces_ExtDirect_ActionHandler
tx_Workspaces_ExtDirect_MassActionHandler
tx_Workspaces_ExtDirect_Server
tx_Workspaces_ExtDirect_ToolbarMenu
Tx_Workspaces_Reports_StatusProvider
tx_Workspaces_Service_AutoPublish
tx_Workspaces_Service_AutoPublishTask
tx_Workspaces_Service_Befunc
tx_Workspaces_Service_Fehooks
tx_Workspaces_Service_GridData
Tx_Workspaces_Service_Stages
tx_Workspaces_Service_Tcemain
tx_Workspaces_Service_Workspaces
tx_Workspaces_Service_WorkspacesTest
Updated by Andreas Kiessling about 13 years ago
I think it depends on where the class is instantiated.
E.g. tx_Workspaces_Service_Fehooks is set as Hook for $GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['tslib/class.tslib_fe.php']['hook_eofe']['workspaces']
-> t3lib_div::callUserFunction($_funcRef,$_params,$this);
which checks if the class is prefixed with user_ or tx_
Have a look at RFC #M14313 in the core list with some discussion and patches for the tx_ vs Tx_ situation
Updated by Georg Ringer almost 13 years ago
Tx_ can be used now, so Tx_ is IMO now the correct CGL for extbase extensions
Updated by Tolleiv Nietsch almost 13 years ago
- Status changed from New to Accepted
Updated by Mr. Hudson over 12 years ago
Patch set 1 of change I6049923420414303c3985b6ff96a80ec1be9ef7f has been pushed to the review server.
It is available at http://review.typo3.org/4466
Updated by Tolleiv Nietsch over 12 years ago
- Status changed from Accepted to Under Review
patch pending in review - TBD whether it's fine to rename classes without deprecation - imho it is.
Updated by Mr. Hudson over 12 years ago
Patch set 2 of change I6049923420414303c3985b6ff96a80ec1be9ef7f has been pushed to the review server.
It is available at http://review.typo3.org/4466
Updated by Tolleiv Nietsch over 12 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset commit:f6df0e62f1bd6dd6f70162c60a8ce47dfc197bb7.
Updated by Xavier Perseguers about 12 years ago
- Status changed from Resolved to Closed
Updated by Michael Stucki over 10 years ago
- Project changed from 624 to TYPO3 Core
- Category changed from Workspaces to Workspaces