Task #52678

Rename API for ApplicationContext

Added by Steffen Müller almost 2 years ago. Updated almost 2 years ago.

Status:Resolved Start date:2013-10-11
Priority:Should have Due date:
Assigned To:- % Done:

100%

Category:- Spent time: -
Target version:6.2.0
TYPO3 Version:6.2 Complexity:
PHP Version: Sprint Focus:

Description

In #50983 we have introduced a public API to get the current application context.

The API was:

\TYPO3\CMS\Core\Utility\GeneralUtility::getContext();

I suggest to change the API name to

\TYPO3\CMS\Core\Utility\GeneralUtility::getApplicationContext();

and use the term applicationContext in all other places.


Related issues

related to Core - Bug #50983: Add public API to get ApplicationContext Resolved 2013-08-10
blocks Core - Feature #50132: TypoScript: Application Context condition Resolved 2013-07-17

Associated revisions

Revision 24f32d1d
Added by Steffen Müller almost 2 years ago

[TASK] Rename API for ApplicationContext

Use the term "applicationContext" instead of "context",
because it is too general and could lead to confusion.

Resolves: #52678
Releases: 6.2
Change-Id: Ib586d9d1eb69e6ca7981ed638d45f7aa5b6e6c70
Reviewed-on: https://review.typo3.org/24541
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn

History

#1 Updated by Steffen Müller almost 2 years ago

  • Subject changed from Rename API to get ApplicationContext to Rename API for ApplicationContext

#2 Updated by Gerrit Code Review almost 2 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/24541

#3 Updated by Gerrit Code Review almost 2 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/24541

#4 Updated by Gerrit Code Review almost 2 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/24541

#5 Updated by Gerrit Code Review almost 2 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/24541

#6 Updated by Gerrit Code Review almost 2 years ago

Patch set 5 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/24541

#7 Updated by Steffen Müller almost 2 years ago

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

Also available in: Atom PDF