Comments on: jQuery “Populate” plugin

By: Jovan Popovic

Jovan Popovic — Sat, 16 Apr 2011 00:05:37 +0000

Hi Dave,

I have tried this plugin and it looks great. I have few questions about the plugin.
1. On the demo page in text area example you wrote that “This functionality is currently buggy, and is undergoing review!”. I have placed textarea and it works fine. What is a bug here?
2. Is there a chance to accept non-string types in JSON. I have check box with values true and false and I need to put something like { Selected: “true” } instead of {Selected: true} in JSON. I don’t see why you have this constraint because to resolve this you will need to change line in the “checkbox” case branch:

element.checked |= element.value == values[j];

element.checked |= element.value == values[j].toString();

I think that the same issue is for select lists. Usually values of select lists are numbers so it is a problem to convert them to string each time, espessially if we are using some library that automatically generate JSON based on actual object.

Regards,
Jovan

By: heinetz

heinetz — Tue, 08 Mar 2011 12:27:53 +0000

… and you ‘ll see nice alerts whenever populating textareas

By: heinetz

heinetz — Tue, 08 Mar 2011 12:26:22 +0000

this is, what it did for me:

#180 case ’select’:
#181 case ’select-one’:
#182 element.value = value.toString() || value;
#183 break;
#184
#185 /*new*/case ‘textarea’:
#186 /*new*/ alert(value);
#187 /*new*/ value = value == null ? ” : value;
#188 /*new*/ element.innerHTML = value;
#189 /*new*/ break;
#190
#191 case ‘text’:
#192 case ‘button’:
#193 /*old*///case ‘textarea’:
#194 case ’submit’:
#195 default:
#196 value = value == null ? ” : value;
#197 element.value = value;

By: heinetz

heinetz — Tue, 08 Mar 2011 11:54:30 +0000

IE6 throws an error, when trying to poplulate the value of a textarea.

best,
heinetz

By: Dave Stewart

Dave Stewart — Tue, 16 Nov 2010 10:47:08 +0000

Thanks Sas. Probably about time I had a maintenance-look at this plugin anyway

By: sas

sas — Thu, 04 Nov 2010 21:30:08 +0000

Hello,
Thanks for the plugin.
I had a problem which arises when the plugin tries to populate form fields that do not exist in the form but do exist in the json/js data array.
for example:
json data elements are: firstname, lastname, email
form fields: firstname, lastname

When the plugin gives js error on IE since it cannot fiend the form field.

Here is a small patch – line 125:
if(element == undefined)
{
debug(’No such element as ‘ + name);
return false;
}

This will ensure that the plugin will return and not try to populate a non-existing form field.

Thanks,

By: Marc

Marc — Fri, 24 Sep 2010 20:50:04 +0000

Just a quick note to say that you’ve helped me learn a better (and cleaner) way to develop web apps.

Cheers!

By: Zane

Zane — Tue, 09 Mar 2010 16:15:24 +0000

Just letting you know that I was getting some weird errors and had to change a line of code:

var parseJSON = function (obj, path)

That was what I changed the parseJSON function line to. It eliminated my issue and didnt seem to introduce any other problems. Just letting you know in case you wanted to add that into your next release. Thanks

By: Dave Stewart

Dave Stewart — Wed, 17 Feb 2010 15:25:32 +0000

Nope – just a use if you want to. I should probably put one in as someone else commented on that. I’m not so anal that I care, but apparently others do

By: jim

jim — Fri, 12 Feb 2010 06:41:32 +0000

Hi Dave,

Is there a license or copyright or something similar that is associated with your code?

Thanks, Jim