Comments on: Using Prototype Javascript to set the value of a radio group http://bluedogwebservices.com/using-prototype-javascript-to-set-the-value-of-a-radio-group/?utm_source=rss&utm_medium=rss&utm_campaign=using-prototype-javascript-to-set-the-value-of-a-radio-group BlueDog Web Services Wed, 09 May 2012 20:39:59 +0000 hourly 1 http://wordpress.org/?v=3.4-beta3 By: Ben Greiner http://bluedogwebservices.com/using-prototype-javascript-to-set-the-value-of-a-radio-group/#comment-124 Ben Greiner Tue, 22 Dec 2009 22:48:58 +0000 http://bluedogwebservices.com/using-prototype-javascript-to-set-the-value-of-a-radio-group/#comment-124 Something else to note.. After doing more testing. I discovered that if the value is already set, it appears that the Aaron's solution does not change the values. In this scenario the radio buttons load run the js script and set their values, GREAT! Now the user comes along and changes the values.. but users being indecisive, decides he doesn't want to change them after all and instead clicks cancel. This will then reset those radio buttons by running the same bit of code, and restoring cached values of the original state. I have discovered that my solution will actually reset those values. Something else to note.. After doing more testing. I discovered that if the value is already set, it appears that the Aaron’s solution does not change the values. In this scenario the radio buttons load run the js script and set their values, GREAT! Now the user comes along and changes the values.. but users being indecisive, decides he doesn’t want to change them after all and instead clicks cancel. This will then reset those radio buttons by running the same bit of code, and restoring cached values of the original state. I have discovered that my solution will actually reset those values.

]]> By: Ben Greiner http://bluedogwebservices.com/using-prototype-javascript-to-set-the-value-of-a-radio-group/#comment-123 Ben Greiner Tue, 22 Dec 2009 22:17:05 +0000 http://bluedogwebservices.com/using-prototype-javascript-to-set-the-value-of-a-radio-group/#comment-123 Oh well now you made me curious.. I did some basic benchmarking. Now on my main Ubuntu machine, it was dead even. 1 millisecond execution time. Not a surprise the thing is a quad core screamer. I ran it with my Dual Core Mac Book Pro and got the follow results.. I had three sets of Option Boxes Each with two Options.. This was with my solution. 1261519984835 1261519984836 1 1261519984906 1261519984908 2 1261519984977 1261519984978 1 With the modified version of your solution.. 1261519929799 1261519929800 1 1261519929868 1261519929869 1 1261519929941 1261519929942 1 So the results are in and your solution has it by 1 millisecond! If anyone else would like to try it out.. function $RFS(radioGroup, value) { var date1 = new Date(); var milliseconds1 = date1.getTime(); //$$("'input[type=radio][name='" + radioGroup + "'][value='" + value + "']")[0].writeAttribute('checked', 'checked'); if(curRadio = $$('input[type=radio]').detect( function isRadioGroup(group){ if(group.getAttribute('id') == $(radioGroup).getAttribute('id') && group.value == value){ return true;} })){ curRadio.checked = true; } var date2 = new Date(); var milliseconds2 = date2.getTime(); var difference = milliseconds2 - milliseconds1; console.log(milliseconds1); console.log(milliseconds2); console.log(difference); } Oh well now you made me curious.. I did some basic benchmarking.

Now on my main Ubuntu machine, it was dead even. 1 millisecond execution time. Not a surprise the thing is a quad core screamer. I ran it with my Dual Core Mac Book Pro and got the follow results..

I had three sets of Option Boxes Each with two Options..

This was with my solution.
1261519984835
1261519984836
1
1261519984906
1261519984908
2
1261519984977
1261519984978
1

With the modified version of your solution..
1261519929799
1261519929800
1
1261519929868
1261519929869
1
1261519929941
1261519929942
1

So the results are in and your solution has it by 1 millisecond!

If anyone else would like to try it out..

function $RFS(radioGroup, value) {
var date1 = new Date();
var milliseconds1 = date1.getTime();

//$$(“‘input[type=radio][name='" + radioGroup + "'][value='" + value + "']“)[0].writeAttribute(‘checked’, ‘checked’);

if(curRadio = $$(‘input[type=radio]‘).detect( function isRadioGroup(group){
if(group.getAttribute(‘id’) == $(radioGroup).getAttribute(‘id’) && group.value == value){ return true;}
})){ curRadio.checked = true; }

var date2 = new Date();
var milliseconds2 = date2.getTime();

var difference = milliseconds2 – milliseconds1;
console.log(milliseconds1);
console.log(milliseconds2);
console.log(difference);

}

]]> By: Ben Greiner http://bluedogwebservices.com/using-prototype-javascript-to-set-the-value-of-a-radio-group/#comment-122 Ben Greiner Tue, 22 Dec 2009 21:53:24 +0000 http://bluedogwebservices.com/using-prototype-javascript-to-set-the-value-of-a-radio-group/#comment-122 But you make an interesting point.. I hadn't thought about delimiting the double quotes and just plug in my vars.. The obvious often eludes me.. $$("'input[type=radio][name=' " + radioGroup + " '][value=' " + value + " ']")[0].writeAttribute('checked', 'checked'); This also works.. But to answer your initial question.. I don't know which version would be faster, though yours is certainly more compact. That would be something interesting to find out certainly.. but alas.. I must be moving on. Think I will use your modified solution for now ;-) But you make an interesting point.. I hadn’t thought about delimiting the double quotes and just plug in my vars.. The obvious often eludes me..

$$(“‘input[type=radio][name=' " + radioGroup + " '][value=' " + value + " ']“)[0].writeAttribute(‘checked’, ‘checked’);

This also works.. But to answer your initial question.. I don’t know which version would be faster, though yours is certainly more compact. That would be something interesting to find out certainly.. but alas.. I must be moving on. Think I will use your modified solution for now ;-)

]]> By: Ben Greiner http://bluedogwebservices.com/using-prototype-javascript-to-set-the-value-of-a-radio-group/#comment-121 Ben Greiner Tue, 22 Dec 2009 21:42:12 +0000 http://bluedogwebservices.com/using-prototype-javascript-to-set-the-value-of-a-radio-group/#comment-121 I was unable to get the multiple selects input[type=radio][name=varName] to return valid values.. Always came back empty. I am using a Prototype class in an external js file that has all the field names passed in as constructor options. When I passed name=this.options.fieldName it seemed to take it as a literal instead of a var. So that would be why I had to do a little work around here.. I was unable to get the multiple selects input[type=radio][name=varName] to return valid values.. Always came back empty. I am using a Prototype class in an external js file that has all the field names passed in as constructor options. When I passed name=this.options.fieldName it seemed to take it as a literal instead of a var. So that would be why I had to do a little work around here..

]]> By: Ben Greiner http://bluedogwebservices.com/using-prototype-javascript-to-set-the-value-of-a-radio-group/#comment-120 Ben Greiner Tue, 22 Dec 2009 21:38:05 +0000 http://bluedogwebservices.com/using-prototype-javascript-to-set-the-value-of-a-radio-group/#comment-120 Ooops Small gaff.. This is the correct function.. My Bad.. function $RFS(radioGroup, value) { if(curRadio = $$('input[type=radio]').detect( function isRadioGroup(group){ if(group.getAttribute('id') == $(radioGroup).getAttribute('id') && group.value == value){ return true;} })){ curRadio.checked = true; } } Ooops Small gaff.. This is the correct function.. My Bad..

function $RFS(radioGroup, value) {
if(curRadio = $$(‘input[type=radio]‘).detect( function isRadioGroup(group){
if(group.getAttribute(‘id’) == $(radioGroup).getAttribute(‘id’) && group.value == value){ return true;}
})){ curRadio.checked = true; }
}

]]> By: AaronCampbell http://bluedogwebservices.com/using-prototype-javascript-to-set-the-value-of-a-radio-group/#comment-119 AaronCampbell Tue, 22 Dec 2009 20:59:35 +0000 http://bluedogwebservices.com/using-prototype-javascript-to-set-the-value-of-a-radio-group/#comment-119 Thanks for the input. Do you think that's going to be as efficient as: [code lang="javascript"]$$("input[type=radio][name='radioGroupName'][value='yourDefaultValue']")[0].writeAttribute("checked", "checked");[/code] Thanks for the input. Do you think that’s going to be as efficient as:

$$("input[type=radio][name='radioGroupName'][value='yourDefaultValue']")[0].writeAttribute("checked", "checked");
]]> By: Ben Greiner http://bluedogwebservices.com/using-prototype-javascript-to-set-the-value-of-a-radio-group/#comment-118 Ben Greiner Tue, 22 Dec 2009 20:50:11 +0000 http://bluedogwebservices.com/using-prototype-javascript-to-set-the-value-of-a-radio-group/#comment-118 Gosh guess I actually have to do the heavy lifting myself here. This is with Prototype 1.8.3.. Though I see no reason why this wouldn't work with older prototype versions. Enjoy ;o)~ [code lang="javascript"] function $RFS(radioGroup, value) { if( curRadio = $$('input[type=radio]').detect( function isRadioGroup(group){ if(group == $(radioGroup) && group.value == value){return true;} })){ curRadio.checked = true; } } [/code] * comment edited by admin to use the `code` shortcode around the source code snippet Gosh guess I actually have to do the heavy lifting myself here. This is with Prototype 1.8.3.. Though I see no reason why this wouldn’t work with older prototype versions. Enjoy ;o)~

function $RFS(radioGroup, value) {
	if( curRadio = $$('input[type=radio]').detect( function isRadioGroup(group){
		if(group == $(radioGroup) && group.value == value){return true;}
	})){ curRadio.checked = true; }
}

* comment edited by admin to use the `code` shortcode around the source code snippet

]]> By: Aaron D. Campbell http://bluedogwebservices.com/using-prototype-javascript-to-set-the-value-of-a-radio-group/#comment-117 Aaron D. Campbell Mon, 16 Jun 2008 18:58:50 +0000 http://bluedogwebservices.com/using-prototype-javascript-to-set-the-value-of-a-radio-group/#comment-117 Rob: Sorry, there was a mistake in that. there should have been a "[0]" following the $$(). Try it with the updated code. Rob: Sorry, there was a mistake in that. there should have been a “[0]” following the $$(). Try it with the updated code.

]]> By: Rob http://bluedogwebservices.com/using-prototype-javascript-to-set-the-value-of-a-radio-group/#comment-116 Rob Mon, 16 Jun 2008 18:46:58 +0000 http://bluedogwebservices.com/using-prototype-javascript-to-set-the-value-of-a-radio-group/#comment-116 It didn't work for me either $$("input[type=radio][name='groupname'][value='myvalue']").writeAttribute is not a function Prototype 1.6.0 It didn’t work for me either

$$(“input[type=radio][name='groupname'][value='myvalue']“).writeAttribute is not a function

Prototype 1.6.0

]]> By: Aaron D. Campbell http://bluedogwebservices.com/using-prototype-javascript-to-set-the-value-of-a-radio-group/#comment-115 Aaron D. Campbell Fri, 01 Feb 2008 17:53:24 +0000 http://bluedogwebservices.com/using-prototype-javascript-to-set-the-value-of-a-radio-group/#comment-115 What version of Prototype were you using? What version of Prototype were you using?

]]>