OK, cool... So I can do...

`
$tags = $_POST['tags'];
$newtag = explode(",", $tags);
foreach($newtag){
`
for the comma seperated tags.

and....

`
if (strcasecmp($newtag, $row['name']) == 0) {
`
for the case insensitive comparison?

`foreach($newtag as $something) {` of course.

I think it might be better to use `strtolower()` before checking as you otherwise might accidentally store double ones in the database later as you have still the value with the case difference.

Good point; I didn't think of that. Whether it's faster to use `strtolower()` or `strcasecmp()` + `array_unique()`, I'm not sure, but `strtolower()` is probably easier.

Also you might wish to have code that will deal with user input strings that might wish to include commas in their tag. (Or you could just tell them that is not allowed.)

I think normally tags are used as "keywords" and rarely include a comma. Of course a more rarely used `;` could be used as a delimiter as well or a `|`.

`
foreach($newtag as $something)
`
Does that mean I have to reference $newtag as $something anywhere else in the loop? Does it change the variable?

Would it be better to use

`
while($newtag = >0) {
`

You probalby need to review your PHP syntax. http://www.php.net/manual/en/control-structures.foreach.php

Also I get a little the impression that you should learn a few more php basics before proceeding. This forum is of course not for learning these...

Understood, I know enough php/mysql to get me in trouble and I have enough tenacity to get what I want. I work at something and learn as I go until it has the desired effect.

I created this on my own and it works fabulously:

`
<?php
mysql_connect("xxx", "xxx", "xxx");
mysql_select_db("xxx");

$album = $_POST['album'];
$tags = $_POST['tags'];

$sqlalbumid = mysql_query("SELECT id FROM zp_albums WHERE folder ='" . $album . "'");
$albumid = mysql_fetch_array( $sqlalbumid );

$newtag = explode(",", $tags);
foreach($newtag as $key){
$otags = mysql_query("SELECT * FROM zp_tags");
$otag = mysql_fetch_array( $otags );
if (strcasecmp($key, $otag['name']) == 0) {
mysql_query("INSERT INTO zp_obj_to_tag (tagid, type, objectid) VALUES ('" . $otag['id'] . "', 'albums', '" . $albumid['id'] . "')");
}
else {
mysql_query("INSERT INTO zp_tags (name) VALUES('" . $key . "')");
$newtagid = mysql_query("SELECT id FROM zp_tags WHERE name ='" . $key . "'");
$tagid = mysql_fetch_array( $newtagid );
mysql_query("INSERT INTO zp_obj_to_tag (tagid, type, objectid) VALUES('" . $tagid['id'] . "', 'albums', '" . $albumid['id'] . "')");
}
}
?>
`
I will hide it behind a password protection and use it, I need to look more at the backend for the admin area and I will make this a plugin.

Thank you for all the feedback.

oops, for my purposes i will need to write something that checks if the folder exists....

Got it...

added:
`
$sqlalbumid = mysql_query("SELECT id, folder FROM zp_albums WHERE folder ='" . $album . "'");
$albumid = mysql_fetch_array( $sqlalbumid );
if (strcasecmp($album, $albumid['folder']) <> 0) {
echo "Album folder " .$album. " does NOT exist, please check the spelling.";
}

else {
`

Two notes:

The code you posted is not secure. You really should be sanitizing anything that comes through `$_POST`. It's probably also not safe against XSRF attacks, though I'm not too familiar with that subject.

I see a lot of `mysql_query()` in your code; it's probably better to compile one long query in your code and then, at the end, actually query the database. (I don't really have experience with databases, but I'm speaking from a coding standpoint.)