Funky textField values when passing a list to select(options=list)

[chrisdpeters]

This is generally what I'm doing with a select in CFWheels 1.4.5 on CF9:

<cfoutput>

#select(
  objectName="foo",
  property="bar",
  options="Website,Website (SP Referral),All AIMS users,FTRI staff,RDC",
  includeBlank="-- Add submitters --"
)#

</cfoutput>

But the list of options generated ends up looking like this:

<option value="">-- All submitters --</option>
<option value="Website">Website</option>
<option value="Website (SP Referral)">Website ( SP Referral)</option>
<option value="All AIMS users">All  AIMSusers</option>
<option value="FTRI staff">FTRIstaff</option>
<option value="RDC">RDC</option>

Notice that the values are OK, but there's weird stuff going on with spaces in the text displayed.

I doubt this makes a difference, but the list is being returned by a function. I don't see why that would matter though.

[kmd1970]

Is there a fix for this in version 1.4.6, for people who haven't made the jump to 2.0 as yet?

[perdjurner]

Yes there is.

[neokoenig]

https://github.com/cfwheels/cfwheels/releases/tag/v1.4.6
"Made humanize() keep spaces in input."

Nice one @perdjurner

[kmd1970]

I tested 1.4.6 before I posted. Looks like the issue remains.

[kmd1970]

FYI, my company firewall blocks the "cfwheels.1.4.6.zip" link for some reason, so I used the "Source code (zip)" link to download 1.4.6. Are they different?

[perdjurner]

File content should be the same (in terms of relevant code anyway).
Perhaps we didn't get the fix done properly then.
Are you able to run that test code you posted on Wheels 2.0 and see if it works?
Just want to make sure which versions are broken exactly.

Thanks.

[kmd1970]

My localhost is not worthy of 2.0. :)
"Adobe ColdFusion 11,0,08,298512 is not supported by CFWheels."

Also, set(disableEngineCheck=true); in settings.cfm doesn't work.
Edit: This worked in URL, /index.cfm?reload=development&disableEngineCheck=true

[kmd1970]

Ok, I was able to verify it's still broken in 2.0. Sorry for the delay.

[kmd1970]

This may fix the issue with Humanize()

// add a space before every capitalized word
loc.rv = REReplace(arguments.text, "([[:upper:]])", " \1", "all");
// remove space after punctuation chars *Added*
loc.rv = REReplace(loc.rv, "([[:punct:]])([[:space:]])", "\1", "all");

Edit: removed the + to handle multiple punctuation characters like in Website (( SP Referral))

[kmd1970]

Found another issue.

//test
#humanize("All AIMS users")#<br>
#humanize("FTRI staff")#<br>

//will output
All AIMSusers
FTRIstaff

Replace

// fix abbreviations so they form a word again (example: aURLVariable)
local.rv = REReplace(local.rv, "([[:upper:]]) ([[:upper:]])(?:\s|\b)", "\1\2", "all");

with

// fix abbreviations so they form a word again (example: aURLVariable)
local.rv = REReplace(local.rv, "([[:upper:]]) ([[:upper:]])(?:\s|\b)(?![a-z])", "\1\2", "all");

//test
#humanize("Website (SP Referral)")#<br>
#humanize("Website ( SP Referral )")#<br>
#humanize("Website (( SP Referral))")#<br>
#humanize("All AIMS users")#<br>
#humanize("FTRI staff")#<br>
#humanize("aURLVariable")#<br>
#humanize("wheelsIsAFramework")#

//will output
Website (SP Referral)
Website ( SP Referral )
Website (( SP Referral))
All AIMS users
FTRI staff
A URL Variable
Wheels Is A Framework 

[perdjurner]

I think my last commit fixed that actually.

…

On Wed, 25 Oct 2017 at 21:35, Bob Marley ***@***.***> wrote: Found another issue. //test #humanize("All AIMS users")#<br> #humanize("FTRI staff")#<br> //will output All AIMSusers FTRIstaff Replace // fix abbreviations so they form a word again (example: aURLVariable)local.rv = REReplace(local.rv, "([[:upper:]])([[:space:]])(?![a-z])", "\1", "all"); with // fix abbreviations so they form a word again (example: aURLVariable)local.rv = REReplace(local.rv, "([[:upper:]]) ([[:upper:]])(?:\s|\b)(?![a-z])", "\1\2", "all"); //test #humanize("Website (SP Referral)")#<br> #humanize("Website ( SP Referral )")#<br> #humanize("Website (( SP Referral))")#<br> #humanize("All AIMS users")#<br> #humanize("FTRI staff")#<br> #humanize("aURLVariable")#<br> #humanize("wheelsIsAFramework")# //will output Website (SP Referral)Website ( SP Referral ) Website (( SP Referral)) All AIMS usersFTRI staffA URL Variable Wheels Is A Framework — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#663 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAJzU1M8Y8E6YK90cAHosB6Kjc7HpDBKks5sv41-gaJpZM4JmnC0> .

[kmd1970]

Oops, you are correct.