abitdodgy / cfw-usermanager

A user manager demo app for ColdFusion on Wheels
23 stars 11 forks source link

Add documentation for ColdFusion and BCrypt.class #10

Closed abitdodgy closed 11 years ago

abitdodgy commented 12 years ago

Currently this is broken on ColdFusion since the code is Railo specific. The code that creates the Java object for BCrypt uses a third argument that specifies the class path. This is Railo only and does not work with ColdFusion.

There are two locations in User.cfc that call CreateObject(). It's probably better to create a function that just returns the BCrypt object, that way any changes will only have to be made to one location.

abitdodgy commented 12 years ago

Fix with commit https://github.com/abitdodgy/cfw-usermanager/commit/7010b9aa49f9cadd8d86ef40ed7b47956d2958d1

abitdodgy commented 12 years ago

Comments should be added to the affected model, User.cfc, right where the call to BCrypt are being made. Should probably simplify the objection creation so it only uses one method to return the BCrypt object.

abitdodgy commented 11 years ago

No longer valid with a17faea8210301fa465f5f017431dffdf66b1fa8