469,575 Members | 1,523 Online
Bytes | Developer Community
New Post

Home Posts Topics Members FAQ

Post your question to a community of 469,575 developers. It's quick & easy.

Mysql column populating drop down menu issue

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
14 3995
dlite922
1,584 Expert 1GB
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
Philth
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
1,584 Expert 1GB
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
Philth
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
6,050 Expert 4TB
@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
Philth
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
6,050 Expert 4TB
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
Philth
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
8,652 Expert Mod 8TB
@Philth
quoted?

..................
Aug 5 '09 #10
Markus
6,050 Expert 4TB
@Dormilich
Yeah, what Dorm said. ;)
Aug 5 '09 #11
Philth
38
Yeah, sorry for being dim today.

Thanks for everyones help.
Aug 5 '09 #12
Dormilich
8,652 Expert Mod 8TB
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
1,584 Expert 1GB
@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
8,652 Expert Mod 8TB
@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.

Similar topics

2 posts views Thread by Jason | last post: by
4 posts views Thread by Mike Wilcox | last post: by
reply views Thread by Mike Chirico | last post: by
9 posts views Thread by elyob | last post: by
3 posts views Thread by Angelos | last post: by
Atli
6 posts views Thread by Atli | last post: by
reply views Thread by suresh191 | last post: by
4 posts views Thread by guiromero | last post: by
By using this site, you agree to our Privacy Policy and Terms of Use.