Address "edit" links on Summary page

I have had a few customers tell me that they had made a mistake during the checkout process and entered the wrong "ship to" address and only noticed it on the checkout "Summary" page. They all asked why there wasn't an easy "edit" link next to "Billing Address" and "Shipping Address". I just kept telling each one that we were working on adding it to the page. I remember that some time ago I had asked about editing the addresses at the summary page and it was mentioned that it would potentially invalidate the shipping rates since shipping is handled during the previous checkout step. Well today I sat down and thought about how to do this and try not to violate any functionality of PC. I think I got it working and I'd like to run it by others here to see if this is an acceptable way to accomplish it.

Before I began I found the .asp files I thought I would need to edit and created backups just incase I really messed things up. There were two files that needed editing, login.asp and OrderVerify.asp.

I added two anchors within login.asp. One is named "billingAddress" and is inserted inbetween lines 1393 and 1394, just before this line of code:

<%response.write dictLanguage.Item(Session("language")&"_order_J")%>&a mp;a mp;l t;/p></td>

Then I added the second anchor named "shippingAddress" further down the page just before this line of code:

<%response.write dictLanguage.Item(Session("language")&"_order_CC")%>& amp; amp; lt;/label>

I saved the file and moved on to OrderVerify.asp. This one was a bit tricker for me since its all trial and error. It wasn't difficult to find where to put links back to the login.asp page, but it wasn't working right the first few times and I kept getting javascript errors. But here's what worked.

In order to add the (edit) link next to the "Billing Address" heading and link it back to the anchor I created previously, I had to change this line of code from:

<p><%=dictLanguage.Item(Session("language")&"_o rderverify_23")%></p>

to

<p><%=dictLanguage.Item(Session("language")&"_o rderverify_23")%> <a href="login.asp?lmode=0#billingAddress">(edit)</a>& amp; amp; lt;/p>

Then I had to add the (edit) link next to the "Shipping Address" heading and link it back to the anchor I created previously. To do this I had to change this line of code from:

response.write("<p>" & dictLanguage.Item(Session("language")&"_orderverify_24") & "</p>")
            end if%>

to

response.write(dictLanguage.Item(Session("language")&"_o rderverify_24" ))
            end if%> <a href="login.asp?lmode=0#shippingAddress">(edit)</a>

What kept giving me problems was the inclusion of the <p> and </p>. It was causing the (edit) link to wrap to the next line instead of being at the end of the heading. Once I figured that out it seemed to work like I had expected.

All this modification does is allow someone to jump back to the two addresses so they can modify them if need be. I know, its just as easy to click the BACK button twice at the bottom of the page, but I think online shoppers have become used to having "edit" buttons right next to the addresses for quick editing. If I have broken any cardinal rules with this modification, please let me know.

I hope this helps others who wish to do the same thing.