PHP post id not working using Javascript variable as ID

Question

I have a site that helps students make schedule's by inputting their classes. When they click a button, they can select the course from a dropdown and enter the class # in a box to the right. I am trying to use php to get the contents of the box, but it's not working. Part of the problem may be that the id of the box is assigned a number (like 1, 2, 3) by javascript.

Here's the JSFIDDLE (only add 1 class, I haven't worked on adding 2 classes yet. ALSO the first number (like 100) is from the dropdown.)

HTML:

<form action='http://people.brandeis.edu/~rnewman/schedule.php' method='POST'>
<div id="boxes"></div>
<br>
<input type='button' value='Add Class' id='add' style='height: 40px;'>
<input type='submit' style='height: 40px;' value='Create Schedule'>
</form>

JQUERY:

var x = 0;
 $(document).ready(function(){
 $("#add").click(function(){

   $('#boxes').append('<input type="text" id="' + ++x + '"><br>');  //the id isn't working
  });
 });

FROM schedule.php

echo $_POST['course'];
echo "<br>";
echo $_POST['0'];
echo $_POST['1'];
echo $_POST['2'];

Please post your code directly in the question, instead of merely linking to another site. Thanks. — user1106925
– user1106925, Commented Aug 6, 2012 at 13:32
That jsfiddle is A LOT of spaghetti code to try to read. Can you post something a little smaller? We just need the relevant code. — Matt
– Matt, Commented Aug 6, 2012 at 13:32
Aren't you assigning the text box's ID attribute? Why not just assign the text box an alpha ID value, like "textbox"? — David Hoerster
– David Hoerster, Commented Aug 6, 2012 at 13:33
@DavidHoerster because eventually there will be multiple textboxes. i need unique ids — user176105
– user176105, Commented Aug 6, 2012 at 13:35
You may also want to assign the identifier to your input's name property, too. I'm not a PHP person, but some other frameworks look for the name property, not the ID. id="textbox' + (++x) + ' name="textbox' + (++x) + '"> — David Hoerster
– David Hoerster, Commented Aug 6, 2012 at 13:38

David Hoerster · Accepted Answer · 2012-08-06 13:43:37Z

2

I'm not a PHP person, but usually frameworks look for an element's name attribute instead of the id attribute. If you add a name attribute to your input text box, it should work:

$('#boxes').append('<input type="text" id="' + ++x + '" name="' + ++x + '"><br>');

Hope this helps!

answered Aug 6, 2012 at 13:43

David Hoerster

28.8k9 gold badges71 silver badges103 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Community · Accepted Answer · 2017-05-23 12:27:33Z

2

As you can see on this question on stackoverflow:

ID and NAME tokens must begin with a letter ([A-Za-z]) and may be followed 
by any number of letters, digits ([0-9]), hyphens ("-"), underscores ("_"), 
colons (":"), and periods (".").

So, in short, you can't have your id's just as numbers, and yes, as others say you should add a name attribute.

edited May 23, 2017 at 12:27

CommunityBot

11 silver badge

answered Aug 6, 2012 at 13:43

Nikola

15.1k21 gold badges108 silver badges171 bronze badges

3 Comments

Nikola Over a year ago

@user176105: I wouldn't be so harsh and just like that go and say not true, without the proper proof to back that up (an official one whould suffice in my book). It might work in your browser, but what about dozens of other browsers (and god forsake the older versions of 'popular' browsers)? As a rule of thumb you can't rely on something what is not defined as a standard and it's something your chose browser supports as a corner case. Trust me I had the same issue when something was working perfectly in chrome but not in firefox.

user176105 Over a year ago

perhaps i was a bit harsh, but the statement "ID and NAME tokens must begin with a letter" is not true. Having it work in one browser proves that statement false.

Nikola Over a year ago

@user176105: yes, but as I said, relaying on "working on one browser" will only bring you into trouble sooner or later with some "weird" browser.

Scott Saunders · Accepted Answer · 2012-08-06 13:43:26Z

0

You should give the input a NAME attribute. That's what PHP will use to index the input.

<input type="text" id="' + ++x + '" name="myNewInputs[]" />

answered Aug 6, 2012 at 13:43

Scott Saunders

30.5k14 gold badges60 silver badges66 bronze badges

1 Comment

Scott Saunders Over a year ago

I just saw David Hoerster comments. His answer should be marked as the correct one, if he adds it.

Collectives™ on Stack Overflow

PHP post id not working using Javascript variable as ID

3 Answers 3

Comments

3 Comments

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

3 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related