Adding New Fields to Form from Database Tab

[axel]

Heh. =] I'm just having the worst luck recently with Form Tools.

Alrighty! So I added a new form to Form Tools, and I had the page already to go and created the form and uploaded it and then it FINALLY dawns on me I forgot a field.

I went into the Form and under the Database tab to add a new field to my form, and I clicked the "Add [ 1 ] Rows" and it took me to the Add New Row page, and there was no new rows. The system just spat out the header where it tells me what to type in each collum, but no new rows.

So I obviously tried to do the "Add New Row" again on that page, and bam! Nothing!

The Page specifically is: http://domain/admin/forms/edit.php on the ADD FIELDS page.

I have no errors in my logs, and I am using 2.0.0-beta-20091213.
Is it just me, or is it everyone else too?
Because if it's just me I'll try to fix it myself, but I think it was with the last two updates.

Thanks!
-Axel

[Ben]

Hey Axel,

Yeah, I'm wondering if something went wrong with an update - things seem to just keep going from bad to worse for you.

What browser & OS are you on? My first guess would be a JS bug, so I'll at least eliminate that possibility before seeing if it's something more severe.

- Ben

[axel]

Yeah I figured. It's just been a bag of crap recently.

I've tested it on Google Chrome and Firefox and IE.
IE tells me "errors on page" when I press "add" on the new rows.

I have no idea. :[

[axel]

Apparently this is what I am getting from Google Chrome. Firefox spits out something like it.

Code:

edit.php:33
Uncaught SyntaxError: Unexpected token ILLEGAL

/global/scripts/add_fields.js:67
Uncaught TypeError: Cannot read property 'phrase_size_tiny' of undefined

(end result of edit.php)
There is ONE part at the end of here where it goes });";
Not sure if that might be the issue? It calls it bad though... :[

Code:

//<![CDATA[
Event.observe(document, 'dom:loaded',
  function()
  {
    add_fields_ns.add_fields("1");
    add_fields_ns.toggle_db_column_fields($('auto_generate_col_names').checked);
  });";

(add_fields.js)
The JS file calls for a command which flips it out all the time with the g.messages command

Code:

option1.appendChild(document.createTextNode(g.messages["phrase_size_tiny"]));

[axel]

Haha! I found it!
Lame thing had me going for 5 hours.

On the page located in:
[formtoolsdirectory]/admin/forms/page_add_fields.php

On line 41 it reads normally:
});";


Correct it to:
});

And it works fine.

[Ben]

Thanks Axel.

I'll release a fix for this today. Alexis reported the same issue; sorry I didn't have time to patch it last night.

- Ben