Task #51482

Epic #70573: Clean up entrypoints

Deprecate script based modules

Added by Kay Strobach about 6 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Could have
Assignee:
Category:
System/Bootstrap/Configuration
Target version:
Start date:
2013-08-29
Due date:
% Done:

100%

TYPO3 Version:
6.2
PHP Version:
5.4
Tags:
Complexity:
Sprint Focus:

Description

Would it make sense to always use the mod.php, instead of using the following line in conf.php?

$MCONF['script']='index.php';

Associated revisions

Revision eef47f94 (diff)
Added by Benni Mack almost 4 years ago

[TASK] Deprecate script-based modules

Deprecate calling addModulePath() and calling modules
via paths.

Resolves: #51482
Releases: master
Change-Id: Ib94b3306ec47f77a907620d838598195e41b51de
Reviewed-on: https://review.typo3.org/43871
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>
Reviewed-by: Wouter Wolters <>
Tested-by: Wouter Wolters <>

History

#1 Updated by Philipp Gampe about 6 years ago

I would even move the stuff in conf.php into the modules registration. IMHO modules registration should be some kind of register plugin with a classname+method.

#2 Updated by Ernesto Baschny about 6 years ago

Would make sense, yes, but the core still does that in some modules:

typo3/sysext/cms/layout/conf.php:$MCONF['script'] = 'db_layout.php';
typo3/sysext/func/mod1/conf.php:$MCONF['script'] = 'index.php';
typo3/sysext/info/mod1/conf.php:$MCONF['script'] = 'index.php';
typo3/sysext/version/cm1/conf.php:$MCONF['script'] = 'index.php';

Please also note that Benni merged this FEATURE some time ago which is related to this: #49643. Since Benni is now "back in business" you might want to get in touch with him for further details or plans in this area.

#3 Updated by Georg Ringer almost 5 years ago

IMO only db_layout is still using the old way...

#5 Updated by Benni Mack over 4 years ago

  • Status changed from New to Accepted
  • Target version changed from 6.2.0 to 7.5

Now we only need to document this behaviour.

#6 Updated by Benni Mack about 4 years ago

  • Category changed from Backend API to 1599
  • Assignee set to Benni Mack
  • Target version changed from 7.5 to 7 LTS

We need to modify EXT:form wizards then we can completely deprecate this logic. Hopefully in time for 7.6

#7 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 http://review.typo3.org/43871

#8 Updated by Gerrit Code Review about 4 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/43871

#9 Updated by Benni Mack about 4 years ago

  • Parent task set to #70573

#10 Updated by Gerrit Code Review almost 4 years ago

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

#11 Updated by Gerrit Code Review almost 4 years ago

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

#12 Updated by Gerrit Code Review almost 4 years ago

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

#13 Updated by Gerrit Code Review almost 4 years ago

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

#14 Updated by Gerrit Code Review almost 4 years ago

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

#15 Updated by Benni Mack almost 4 years ago

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

#16 Updated by Riccardo De Contardi almost 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF