Skip to content

Commit d50ac43

Browse files
committed
* Adobe 2016 Dropped
1 parent 44dae7f commit d50ac43

File tree

8 files changed

+10
-60
lines changed

8 files changed

+10
-60
lines changed

.github/workflows/tests.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ jobs:
1717
strategy:
1818
fail-fast: false
1919
matrix:
20-
cfengine: [ "lucee@5", "adobe@2016", "adobe@2018", "adobe@2021" ]
20+
cfengine: [ "lucee@5", "adobe@2018", "adobe@2021" ]
2121
steps:
2222
- name: Checkout Repository
2323
uses: actions/checkout@v2

ModuleConfig.cfc

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ component {
77

88
// Module Properties
99
this.title = "ColdBox Debugger";
10-
this.author = "Curt Gratz - Ortus Solutions";
10+
this.author = "Ortus Solutions";
1111
this.version = "@build.version@[email protected]@";
1212
this.webURL = "https://www.ortussolutions.com";
1313
this.description = "The ColdBox Debugger Module";
@@ -262,10 +262,7 @@ component {
262262

263263
/******************** ACFSQL COLLECTOR ************************************/
264264

265-
// Do not load on lucee or ACF 2016
266-
if (
267-
variables.settings.acfSql.enabled && !server.keyExists( "lucee" ) && server.coldfusion.productVersion.listFirst() gt "2016"
268-
) {
265+
if ( variables.settings.acfSql.enabled && !server.keyExists( "lucee" ) ) {
269266
interceptorService.registerInterceptor(
270267
interceptorClass = "#moduleMapping#.interceptors.ACFSqlCollector",
271268
interceptorName = "ACFSqlCollector@cbdebugger"

box.json

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -60,15 +60,12 @@
6060
"install:2021":"run-script cfpm:install zip,debugger,orm,mysql,postgresql,sqlserver,feed,chart",
6161
"install:dependencies":"install && cd test-harness && install",
6262
"start:lucee":"server start [email protected]",
63-
"start:2016":"server start [email protected]",
6463
"start:2018":"server start [email protected]",
6564
"start:2021":"server start [email protected]",
6665
"stop:lucee":"server stop name=cbdebugger-lucee@5",
67-
"stop:2016":"server stop name=cbdebugger-adobe@2016",
6866
"stop:2018":"server stop name=cbdebugger-adobe@2018",
6967
"stop:2021":"server stop name=cbdebugger-adobe@2021",
7068
"logs:lucee":"server log name=cbdebugger-lucee@5 --follow",
71-
"logs:2016":"server log name=cbdebugger-adobe@2016 --follow",
7269
"logs:2018":"server log name=cbdebugger-adobe@2018 --follow",
7370
"logs:2021":"server log name=cbdebugger-adobe@2021 --follow"
7471
},

build/.travis.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ env:
1010
- MODULE_ID=cbdebugger
1111
matrix:
1212
- ENGINE=lucee@5
13-
- ENGINE=adobe@2016
1413
- ENGINE=adobe@2018
1514

1615
branches:

changelog.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
77

88
----
99

10-
## [3.5.0] => 2022-SEP
10+
## [4.0.0] => 2022-SEP
1111

1212
### Added
1313

@@ -33,6 +33,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
3333
* Tracers are now streamlined and stored alongside the request profilers
3434
* Small UI fixes on request profiler HTTP methods
3535
* WireBox collecting is now done by the WireBox collector not the Request Collector.
36+
* Adobe 2016 Dropped
3637

3738
----
3839

interceptors/RequestCollector.cfc

Lines changed: 1 addition & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ component extends="coldbox.system.Interceptor" {
9393
// Don't render in ajax calls
9494
!arguments.event.isAjax() AND
9595
// Only show on HTML content types
96-
findNoCase( "html", getPageContextResponse().getContentType() ) AND
96+
findNoCase( "html", variables.debuggerService.getPageContextResponse().getContentType() ) AND
9797
// We don't have any render data OR the render data is HTML
9898
(
9999
structIsEmpty( arguments.event.getRenderData() ) || arguments.event.getRenderData().contentType == "text/html"
@@ -259,19 +259,6 @@ component extends="coldbox.system.Interceptor" {
259259
}
260260
}
261261

262-
/**
263-
* Helper method to deal with ACF2016's overload of the page context response, come on Adobe, get your act together!
264-
**/
265-
private function getPageContextResponse(){
266-
var response = getPageContext().getResponse();
267-
try {
268-
response.getStatus();
269-
return response;
270-
} catch ( any e ) {
271-
return response.getResponse();
272-
}
273-
}
274-
275262
/**
276263
* Execute Debugger Commands
277264
*/

models/DebuggerService.cfc

Lines changed: 4 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -499,20 +499,12 @@ component
499499
}
500500

501501
/**
502-
* Helper method to deal with ACF2016's overload of the page context response, come on Adobe, get your act together!
502+
* Helper method to deal with ACF's overload of the page context response, come on Adobe, get your act together!
503503
*/
504504
function getPageContextResponse(){
505-
if ( server.keyExists( "lucee" ) ) {
506-
return getPageContext().getResponse();
507-
}
508-
if ( server.coldfusion.productVersion.listFirst() eq 2016 ) {
509-
return getPageContext()
510-
.getResponse()
511-
.getResponse()
512-
.getResponse();
513-
} else {
514-
return getPageContext().getResponse().getResponse();
515-
}
505+
return server.keyExists( "lucee" ) ? getPageContext().getResponse() : getPageContext()
506+
.getResponse()
507+
.getResponse();
516508
}
517509

518510
/**

[email protected]

Lines changed: 0 additions & 23 deletions
This file was deleted.

0 commit comments

Comments
 (0)