search

teamcfadvance / ValidateThis

An object oriented validation framework for ColdFusion objects
http://www.validatethis.org
44 stars 32 forks source link

ColdBox Interceptor on UNIX w/ ColdBox 3 #84

Closed orangexception closed 11 years ago

orangexception commented 11 years ago

The ColdBox interceptor uses the older style of ColdBox notation. They changed the "interceptor" and "logger" filenames from starting with lowercase to uppercase letters.

On UNIX this causes problems such as Could not find the ColdFusion component or interface coldbox.system.plugins.logger.

Suggest adding a new interceptor for recent ColdBox framework and keeping original for older version of framework.

aliaspooryorik commented 11 years ago

We could just remove the logger calls - they aren't really need - would that do the trick?

orangexception commented 11 years ago

Not sure how is best to solve it. It may make sense to have a separate interceptor for ColdBox 3x series. Really, there's only two changes.

ColdBox 2x => 3x

  1. getPlugin("logger") => getPlugin("Logger")
  2. extends="coldbox.system.interceptor" => extends="coldbox.system.Interceptor"
aliaspooryorik commented 11 years ago

I think it should be upgraded to support ColdBox 3 and people on ColdBox 2 can just updated the path cases if required. ColdBox 3.0.0 was released almost 2 years ago.

aliaspooryorik commented 11 years ago

fixed in https://github.com/ValidateThis/ValidateThis/commit/330ef8b026f2296448813619b60deceac6b39a19