Race-time condition from core layouts.cfm about cfwheels HOT 8 OPEN

layouts.cfm hadn't been changed in years until the syntax updates ticket that @chapmandu worked on.

I reverted back the syntax updates, I can't imagine how that would cause this bug but so far I haven't seen the error again so that's promising. I just have no idea how these changes would cause this issue.

Here's the commit I reverted back from:
7dabfd4

from cfwheels.

Nevermind, it's back 🤦🏻‍♂️

from cfwheels.

@neokoenig Any ideas? Is it possible the array could be modified somewhere else concurrently which would account for why CF counts 4 items in the array on one line then on the next only counts 3?

from cfwheels.

@neokoenig Any ideas? Is it possible the array could be modified somewhere else concurrently which would account for why CF counts 4 items in the array on one line then on the next only counts 3?

Not off the top of my head, but just wondering if it is actually a "blank" layout, then you could use usesLayout(false)? (might not be of course)

from cfwheels.

Thanks Tom! Fortunately, the layout file is actually blank. It just has <cfoutput>#includeContent()#</cfoutput>.

Just updated the code in prod, praying that fixes it 🙏

from cfwheels.

@neokoenig Ugh - Similar to the last error, except now it's saying Array index [5] out of range, array size is [4]

If you look at the code, this error should not be possible. Any other ideas?

from cfwheels.

I'm going to try changing the way we loop over the layouts array. Maybe that'll help:

from cfwheels.

@timsayshey Is this issue fixed?

from cfwheels.