How to do a onchange event on text field that is part of a repeated region

**jhardman** · Oct 1 '09, 10:14 PM

The problem is that all of these repeated items have the same name and ID. include an (x) in the name and ID, and also when you call the javascript function. Hope this helps.

Jared

**kirkgilbert** · Oct 2 '09, 12:19 AM

Thank for the quick reply. I am very new to ASP, so I am a little confused by what you are saying. Could you please show me what you are referring to

**jhardman** · Oct 5 '09, 07:08 PM

each control on the page needs to have a unique name and ID. So if they each have the same name, the javascript function will not be able to tell which box you are trying to update. Does that make sense?

so when you have this:

Code:

 <input name="REVUNPRCE" type="text" id="REVUNPRCE3"
       onChange='docopycat(this.id)' value="<%= FormatNumber((FO.Fields.Item("LI").Value), 2, -2, -2, -2) %>"  size="4">

notice that every repeat has the same name and ID. This is what is causing the problem. In order to sort it out, you will need to give the control an ID and Name that are different for each repeat, I usually do this like this:

Code:

for x = 1 to 25
   '...
   <input type="text" name="textbox<%=x%>">
   '...
next

Does that make sense? Now your loop is going through a db table, so the most logical solution would be some field from the db like an ID number. that way each row in the db is referenced in a single line in the web page made from this asp page.

Jared

**kirkgilbert** · Oct 5 '09, 07:40 PM

I understand exactly what you are saying, I am just not sure where to apply what you are saying. As stated earlier I am new to ASP, so I am working my way through different projects trying to get the hang of it. Could you please so me in the code I have provide where to apply the loop

**jhardman** · Oct 5 '09, 09:18 PM

OK, first, you have a loop in place. Everywhere from "while" to "wend" is a loop, this is the repeated part of your page. There are several ways to make a loop, your "while...we nd" is one of them, you could probably acheive the same results with "for each ... next" or "for x = 1 to 100... next" or "do... loop". You could probably get similar results from each of these. I tend to use "do ... loop"s in this case, it's easy to say "do until fo.eof ... loop" (keeping looping until you get to the end of the table), but since that part is working, I suggest you don't break it. The one part I would change is in the "select" statement, the db query. You say

Code:

SELECT concat(xcust,char(order)),XCUST,ORDDTE,DATED,PROGDL,DATED,CHAR(ORDER) AS ORDER,1-1 AS LI,LINENUM,ITEMNO,DESC,SHPQTY,LNDISC,SHPDTE, UNPRCE,OVTERM,PROGCD,OVTERM,MEDCMF99.NAME,ADDR,CITY,STATE,ZIP,EXT,(UNPRCE/(1.0-(LNDISC*.01))) AS GPRICE, (UNPRCE*(1.0-(OVTERM*.01))) AS GPRIC,concat(Trim(char(stdpck)) ,spcdsc)as STDPCK,NETOHQTY,ONPOQTY,PODELDAT  FROM QS36F.ORDERFILE,MEDCMF99  WHERE concat(xcust,Char(ORDER)) = '0109430046252'  AND XCUST=CUSTNO

You didn't name this first field you are pulling up, and I think you should use it as an ID, so I would say this

Code:

SELECT concat(xcust,char(order)) as ID,...

That gives this first field a name that will be different for each row of the recordset. Then in the input, I would add this ID to the name of the input, so that each repeated input has its own name:

Code:

 <input name="REVUNPRCE_<%=FO("ID")%>" type="text" id="REVUNPRCE_<%=FO("ID")%>"
         onChange='docopycat(this.id)' value="<%= FormatNumber(FO("LI"), 2, -2, -2, -2) %>"  size="4">

By the way, the shorthand FO("ID") is the same as writing FO.Fields.Items ("ID").Value

Does this make sense?

Jared

**kirkgilbert** · Oct 6 '09, 12:10 PM

I made the changes that you suggested. Without applying the loop the page works similar as before. If I have 2 rows of data and lets say I update the second row it is doing the onchange event of row 1. I am still not sure where to apply the loop.

Code:

 <input name="REVUNPRCE_<%=FO("ID")%>" type="text" id="REVUNPRCE_<%=FO("ID")%>" onChange="docopycat(this.id)" value="<%= FormatNumber((0.00),2,-2,-2,-2)%>" size="4">

Would it be right after the code displayed above and if so, everytime I ran the page I received an error. And thank you for you patience in this matter.

**jhardman** · Oct 7 '09, 03:25 PM

Originally posted by kirkgilbert

Would it be right after the code displayed above and if so, everytime I ran the page I received an error. And thank you for you patience in this matter.

I tried to tell you already, you already have a loop. Please stop going on about a loop. I'm sorry I mentioned it. I brought it up because I thought it would help you understand the code you already had.

The issue of the wrong row being affected is still basically the same issue. When you make the change you are still telling the javascript function to change the first row (so you will need to make it change the row you are on) or you are telling it to change a field that has the same name as a hundred other fields.

You need to

make sure the fields all have different names (don't just look through the ASP code, open the HTML source code from your browser and scroll down through it, you will find it informative) and
make sure the javascript function is trying to update the row you are on

I could try to work through your code and throw a code solution at you, but that wouldn't help you understand what is going on. In order to restore my faith in humanity, please try these two things above and see if you can see what is happening. Then if you have a question about how to fix it, I will be glad to help.

Jared

How to do a onchange event on text field that is part of a repeated region

How to do a onchange event on text field that is part of a repeated region

Comment

Comment

Comment

Comment

Comment

Comment

Comment