By using this site, you agree to our updated Privacy Policy and our Terms of Use. Manage your Cookies Settings.
435,144 Members | 880 Online
Bytes IT Community
+ Ask a Question
Need help? Post your question and get tips & solutions from a community of 435,144 IT Pros & Developers. It's quick & easy.

Mysql column populating drop down menu issue

P: 38
Hi there,

I've essentially got a form with several drop down, each populated by columns in various tables.

The populating bit works fine - the column rows appear as they should in the menu.

Ideally the user needs to make their selections, and enter the form into a new database table.

The problem is, when the selection is entered into the new table, only the first word of the string is entered. For example, the drop down may have...

"Category One"
"Category Two"
"Category Three"

When the form is submitted only the word "Category" is entered into the database.

Here is the code, it takes the column 'courseTitle' from one table, displays it in a menu, and then saves the selection into a new table as 'eventName'.

Expand|Select|Wrap|Line Numbers
  1. <?
  2. include("connect.php"); 
  3. $query="SELECT courseTitle FROM course";
  4. $result = mysql_query ($query);
  5. echo "<select name='eventName'=''></option>";
  6. while($nt=mysql_fetch_array($result)){//Array or records stored in $nt
  7. echo "<option value=$nt[courseTitle]>$nt[courseTitle]</option>";
  8. }
  9. echo "</select>";// Closing of list box 
  10. ?>
  11.  

Apologies if I've made it slightly long winded.
Aug 4 '09 #1
Share this Question
Share on Google+
14 Replies


dlite922
Expert 100+
P: 1,584
I don't know why you have an equal sign and extra quotes here:

Expand|Select|Wrap|Line Numbers
  1. "<select name='eventName'=''></option>";
  2.  
but debug your post vars like this
Expand|Select|Wrap|Line Numbers
  1.  die(print_r($_POST))
and see if the data gets there fine.

If that's fine, look in your insert SQL (not shown in your code). Just print it and see if it looks right.

If you don't find anything, post that code back here and any other notes/comments.

Apologies if I've made it slightly long winded.
Nope, it's perfect. I wish every user provided this concise amount of info.



Dan
Aug 4 '09 #2

P: 38
Many thanks for the reply Dan,

I didn't notice the extra quotes. They made no difference either way.

The insert SQL is nothing special, I can't spot anything wrong with it.

Expand|Select|Wrap|Line Numbers
  1. <? 
  2.  
  3. if($_POST['submit'])  
  4. {
  5. include("connect.php"); 
  6.  
  7.    $eventName = $_POST['eventName'];
  8.    $eventCategory1 = $_POST['eventCategory1'];
  9.    $eventCategory2 = $_POST['eventCategory2'];
  10.    $eventCategory3 = $_POST['eventCategory3'];   
  11.    $eventLocation = $_POST['eventLocation'];
  12.    $eventDate = $_POST['eventDate'];   
  13.  
  14.    $result=MYSQL_QUERY("INSERT INTO events (eventName, eventCategory1, eventCategory2, eventCategory3, eventLocation, eventDate)".
  15.       "VALUES ('$eventName', '$eventCategory1', '$eventCategory2', '$eventCategory3', '$eventLocation', '$eventDate')"); 
  16.  
  17.    echo "New Course Event Added"; 
  18. }
  19. ?>
  20.  
Aug 4 '09 #3

dlite922
Expert 100+
P: 1,584
echo $eventName right before the mysql_query() call, and print it's output. I don't know if you know this, but here's how I check my variables when I debug:

Expand|Select|Wrap|Line Numbers
  1. die(var_dump($eventname));  
Another thing I can think of is your MySQL table definition. perhaps the field name is not long enough. ALTER TABLE and change that field to VARCHAR(100).

FYI: You can do "desc events" to give you a definition of the table from a mysql command prompt, if you have access to one.

Let me know,



Dan
Aug 4 '09 #4

P: 38
I added the debug line on line 13 of the second script I posted - it returned the value "Null". The field is set to VARCHAR(100).

Was line 13 the right place to put it?
Aug 5 '09 #5

Markus
Expert 5K+
P: 6,050
@Philth
That's because you probably did $eventname where it should be $eventName.

Anyway, var_dump() your POST array, to check it's being populated correctly by your form.

Expand|Select|Wrap|Line Numbers
  1. var_dump($_POST);
  2. // or
  3. print_r($_POST);
  4.  
Also, with your mysql_query()ies, while debugging, you're provided the function mysql_error() which is populated with a description of the latest mysql error (duh). So, as is popular, we append 'or die(mysql_error());' to any mysql queries - some people don't like the 'or die()' statements, but it's purely preferential.

Expand|Select|Wrap|Line Numbers
  1. mysql_query($query) or die(mysql_error());
  2.  
Mark.
Aug 5 '09 #6

P: 38
Mark,

Thanks for the reply.

The var_dump gave the following info...

array(7) { ["eventName"]=> string(6) "Master" ["eventCategory1"]=> string(4) "VHDL" ["eventCategory2"]=> string(4) "VHDL" ["eventCategory3"]=> string(4) "VHDL" ["eventLocation"]=> string(7) "Germany" ["eventDate"]=> string(0) "" ["submit"]=> string(13) "Add New Event" }

Event name should read "Master VHDL Effectively".

I'm seriously confused.
Aug 5 '09 #7

Markus
Expert 5K+
P: 6,050
Ah, I see the problem: If you do not use quotes around an HTML attribute's value, it'll only read up to the first space as the value. Consider the following:

Expand|Select|Wrap|Line Numbers
  1. <input value=my name is mark name=name />
  2. // $_POST['name'] will hold the value 'my'.
  3. <input value="my name is mark" name="name" />
  4. // $_PSOT['name'] will hold "my name is mark"
  5.  
Always surround attribute values with quotes.

Mark.
Aug 5 '09 #8

P: 38
Hi Mark,

Apologies for not understanding. If the issue is to do with the values before being posted, then the error must be with the first script that I posted?

Expand|Select|Wrap|Line Numbers
  1. <?
  2. include("connect.php"); 
  3. $query="SELECT courseTitle FROM course";
  4.  
  5.  
  6. $result = mysql_query ($query);
  7. echo "<select name='eventName'></option>";
  8.  
  9. while($nt=mysql_fetch_array($result)){
  10. echo "<option value=$nt[courseTitle]>$nt[courseTitle]</option>";
  11.  
  12. }
  13. echo "</select>";
  14. ?>
  15.  
All HTML values are quoted.
Aug 5 '09 #9

Dormilich
Expert Mod 5K+
P: 8,639
@Philth
quoted?

..................
Aug 5 '09 #10

Markus
Expert 5K+
P: 6,050
@Dormilich
Yeah, what Dorm said. ;)
Aug 5 '09 #11

P: 38
Yeah, sorry for being dim today.

Thanks for everyones help.
Aug 5 '09 #12

Dormilich
Expert Mod 5K+
P: 8,639
and while we're quoting......... you should always quote the array's keys (unless they're variables or indeed constants (which is btw rather rare))
Aug 5 '09 #13

dlite922
Expert 100+
P: 1,584
@Philth
Me too, I knew it has something to do with the values not posting correctly, thus form issue.

I always quote my html attributes. It's best to always ensure they're valid also:

http://validator.w3.org/

preferably with XHTML 1.0

Cheers,



Dan
Aug 5 '09 #14

Dormilich
Expert Mod 5K+
P: 8,639
@dlite922
sadly, there is no XHTML support in IE.......
Aug 5 '09 #15

Post your reply

Sign in to post your reply or Sign up for a free account.