History | Log In     View a printable version of the current page.  
Issue Details (XML | Word | Printable)

Key: FC-1387
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Blocker Blocker
Assignee: Matthew Bryant
Reporter: Joel Cass
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
FarCry Core Framework

Permissions Issue - Webskin templates are case sensitive (fix attached)

Created: 01/Jul/08 11:14 AM   Updated: 02/Nov/08 06:02 PM
Component/s: Installer & Updaters
Affects Version/s: FarCry 5.0 (Fortress)
Fix Version/s: FarCry 5.1

Time Tracking:
Not Specified


 Description  « Hide
We were getting erratic permissions issues after migrating a site from FarCry 4.0.10 or 5.0.0

It seems that the permissions system is case sensitive. Strangely enough all the templates seem to start with lowercase "display", however TitleCase "Display" is being passed to the "checkWebskin" method. I'm sure this is something to do with the migration we did and not with FarCry 5.0 itself, however I see that it would be likely for a similar issue to crop up for other users in the future.

After about 1 hour of shuffling through code, I found that a regular expression was case sensitive.

/farcry/core/packages/types/farRole.cfc line 248:

{code}
<cfif (not find(".",filter) or listfirst(filter,".") eq "*" or listfirst(filter,".") eq arguments.type) and refind(replace(listlast(filter,"."),"*",".*","ALL"),arguments.webskin)>
{code}

Replace with

{code}
<cfif (not find(".",filter) or listfirst(filter,".") eq "*" or listfirst(filter,".") eq arguments.type) and reFindNoCase(replace(listlast(filter,"."),"*",".*","ALL"),arguments.webskin)>
{code}


 All   Comments   Work Log   Change History      Sort Order: Ascending order - Click to sort in descending order
Geoff Bowers - 05/Jul/08 03:50 PM
Thanks for lodging the bug. I'm not sure why its a bug though? Isn't case sensitivity an important requirement for file references on non-Windows systems?

Joel Cass - 07/Jul/08 09:22 AM
I don't think this is a windows / non-windows issue, all of our templates always start with a lowercase "display", yet in the database some were set as "Display", e.g. "DisplayTeaserStandard" instead of "displayTeaserStandard"

I'm not sure of the source of this issue exactly, but if we have gotten the problem after upgrading it may be likely that others have it too.

Matthew Bryant - 02/Nov/08 06:02 PM
Thanks for fix.