any chance that #lname# has an apostrophe in it?  like O'Hara or
something similar?

regardless, might want to throw a JSStringFormat() around those values.

can you paste the code from viewing source in the browser?  easier to
debug the rendered code.


On 11/2/05, Orlini, Robert <[EMAIL PROTECTED]> wrote:
> I get this JavaScript error on the mouseover below: Expected ')'
>
> <tr onMouseover="HighLightTR('##f0f0e1');" 
> ondblclick="self.location=('getinventory4.cfm?id=#id#&lname=#lname#&fname=#fname#&serialnumber=#serialnumber#');">
>
> When I just use the ?id=#id# after getinventory4.cfm I do not get an error, 
> but I need to pass the other parameters as well.
>
> What am I missing here please?
>
> Thanks as always!
>
> Robert O.
> HW
>
> Rest of initial code:
>
> <script>
> //+-+-+-+-+-+-+-+-+-+-+-+-+for highlight row+-+-+-+-+-+-+-+-+-+-+-+-+//
> //desc: Highligt current row by click any cell
> //by: Qing-Hua Jiang
> //argument: backColor - highlight bgColor of the row
> // textColor - lighlight text color
> //call: like <tr onClick="HighLightTR('#c9cc99','cc3333');" id="trO4">
>
> var preEl ;
> var orgBColor;
> var orgTColor;
> function HighLightTR(backColor,textColor){
> if(typeof(preEl)!='undefined') {
> preEl.bgColor=orgBColor;
> try{ChangeTextColor(preEl,orgTColor);}catch(e){;}
> }
> var el = event.srcElement;
> el = el.parentElement;
> orgBColor = el.bgColor;
> orgTColor = el.style.color;
> el.bgColor=backColor;
>
> try{ChangeTextColor(el,textColor);}catch(e){;}
> preEl = el;
> }
> function ChangeTextColor(a_obj,a_color){  ;
> for (i=0;i<a_obj.cells.length;i++){//put condition before increase!!!!!
> a_obj.cells(i).style.color=a_color;
> }
> }
>
> //+-+-+-+-+-+-+-+-+-+-+-+-+End of highlight row+-+-+-+-+-+-+-+-+-+-+-+-+//
> </script>
>
>
>
> --
> No virus found in this outgoing message.
> Checked by AVG Free Edition.
> Version: 7.1.362 / Virus Database: 267.12.7/156 - Release Date: 11/2/2005
>
>
>
> 

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~|
Discover CFTicket - The leading ColdFusion Help Desk and Trouble 
Ticket application

http://www.houseoffusion.com/banners/view.cfm?bannerid=48

Message: http://www.houseoffusion.com/lists.cfm/link=i:4:222966
Archives: http://www.houseoffusion.com/cf_lists/threads.cfm/4
Subscription: http://www.houseoffusion.com/lists.cfm/link=s:4
Unsubscribe: 
http://www.houseoffusion.com/cf_lists/unsubscribe.cfm?user=11502.10531.4
Donations & Support: http://www.houseoffusion.com/tiny.cfm/54

Reply via email to