[#10845] - Deprecate the $_PROFILER global var
- Fixed in Code Base
- 14 Aug 2016
- Medium
- Build: staging
- # 10845
- Diff
- mbabker:deprecate-global-profiler
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
- Success JTracker/HumanTestResults Human Test Results: 2 Successful 0 Failed. Details
User tests: Successful: Unsuccessful:
Summary of Changes
Deprecates the $_PROFILER global variable. The profiler should be accessed via the JProfiler API. Replaced existing uses of the global variable with locally scoped versions.
Testing Instructions
While in debug mode, the application profiler should continue to work normally.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
|
||
| Category | ⇒ | Libraries |
Yep. Didn't find that one as my search was only for the variable and not a
structure like that.
On Thursday, June 16, 2016, andrepereiradasilva notifications@github.com
wrote:
tesed with sucess
but notice that _PROFILER also exists in
https://github.com/joomla/joomla-cms/blob/staging/components/com_finder/views/search/view.html.phpShouldnt it be changed here too?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#10845 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AAWfoWVY3CSV9JA1SzgyHpID7OGhfqtZks5qMchfgaJpZM4I32eN
.
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10845.
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10845.
| Status | Pending | ⇒ | Ready to Commit |
| Labels |
Added:
?
|
||
| Category | Libraries | ⇒ | Administration Front End Components Libraries Plugins |
rdeutz
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-08-14 16:53:12 |
| Closed_By | ⇒ | rdeutz |
| Labels |
Removed:
?
|
||
tested with success
but notice that _PROFILER also exists in https://github.com/joomla/joomla-cms/blob/staging/components/com_finder/views/search/view.html.php
Shouldnt it be changed here too?