Skip to main content
P
pauljarema
Participant
March 3, 2023
Answered

Possible Bug: Setting currentRow variable

  • March 3, 2023
  • 2 replies
  • 1270 views

My organization recently updated from CF2018 to CF2021 and I've encountered some odd behavior for which Google has been unable to provide an answer. The following line of code started throwing an error:

<cfset queryName.currentRow += queryName2.numberOfLines>

The error is:

There is a problem in the column mappings specified in the columnMap structure.The query attribute input does not contain any column by the name of currentRow.

Even a simpler test throws the same error:

<cfset queryName.currentRow = '10'>

 I cannot figure out if this was an intentional change between CF2018 and CF2021 or a bug. I do know that it was not throwing an error in CF2018. Has anyone encountered something like this?

    This topic has been closed for replies.
    Correct answer BKBK

    ColdFusion 2021 is actually giving you a hint. QueryName.currentRow is actually a system property in ColdFusion. As such it already has a value. You can confirm this using the following line of code:

    <p>Row number = <cfoutput>#queryName.currentRow#</cfoutput></p>

    It is therefore wrong for the programmer to assign a value to queryName.currentRow. It just might be that you're seeing the error now because CF2021 is in general more accurate than CF2018.

      

    2 replies

    D
    Dordrecht7177366
    Inspiring
    August 14, 2024

    Thank you all for the explanation of the CF2021 issue explained in this thread. I am experiencing the same issue. I just performed a simple query, tried to set the recordcount afterward, and got the same error.

     

    My question is about whether there might be a JVM argument to revert this back to the way things worked in CF2018, or some other global fix. Our codebase appears to set recordcount manually upwards of 90 times throughout, and our application is already live. Any info would be greatly appreciated.

    BKBK
    Community Expert
    BKBKCommunity Expert
    Community Expert
    August 14, 2024

    @Dordrecht7177366 , Could you share the code and the stacktrace of the error?

    D
    Dordrecht7177366
    Inspiring
    August 14, 2024

    Hi, BKBK. 


    The error is exactly the same as pauljarema's above, and I've tested it and confirmed that the issue is due to setting the recordcount of query results to zero with the cfset tag. Previous coders of our app have code that sets query recordcounts to 0 in cases where the query results don't matter, followed by conditionals that take action only if the recordcount is greater than 0. 

     

    Recoding all of the instances to use other variables would be a big task, with a great deal of testing required, which is not desired because the ColdFusion app is being rewritten in a different language over the coming year anyway.

    BKBK
    Community Expert
    BKBKCommunity ExpertCorrect answer
    Community Expert
    March 6, 2023

    ColdFusion 2021 is actually giving you a hint. QueryName.currentRow is actually a system property in ColdFusion. As such it already has a value. You can confirm this using the following line of code:

    <p>Row number = <cfoutput>#queryName.currentRow#</cfoutput></p>

    It is therefore wrong for the programmer to assign a value to queryName.currentRow. It just might be that you're seeing the error now because CF2021 is in general more accurate than CF2018.

      

    P
    pauljaremaAuthor
    Participant
    March 6, 2023

    @BKBK Agree with your points.

     

    My confusion stemmed from the fact that the code in my example worked fine without error (even if it was not correct) until CF was upgraded. If this was a flaw in CF2018 that was fixed in CF2021, I couldn't find anything in the release notes stating that.

     

    If the error I'm getting now IS actually the intended behavior, I'm willing to accept that and implement a workaround to what my example is attempting to accomplish.

    Paul JaremaAdobe Certified Professional: ColdFusion
    BKBK
    Community Expert
    BKBKCommunity Expert
    Community Expert
    March 7, 2023

    Hi @pauljarema ,

    Your confusion is understandable. ColdFusion 2018 should have flagged the assignment

     

    <cfset queryName.currentRow = '10'>

     

    as an error. In any case, that line would have produced incorrect results! 😞

     

    Although CF2018 tolerates the line, the value of queryName.currentRow is not necessarily 10, as you would expect. If the current row is 1, then the value of  queryName.currentRow will be 1. Irrespective of the assignment. In other words, the 10 is ignored. 

     

    That is the behaviour in CF10, CF11, CF2016 and CF2018. In fact, in these CF versions, the following code works, with 1 as output.

     

    <cfscript>
queryName = queryNew("id,name","Integer,Varchar", 
[ 
{id=1,name="One"}, 
{id=2,name="Two"}, 
{id=3,name="Three"} 
]); 

queryName.currentrow="abracadabra";
writedump(queryName.currentrow);
</cfscript>

     

     

    Go to TryCF and see for yourself.

    Terms & ConditionsAccessibility statement
    Using the community Terms of use Privacy Cookie preferences Do not sell or share my personal information ad choicesAdChoices