Jump to content

Recommended Posts

Hi.
I am making a page where I can edit my links.
I have created two buttons which when clicked moves the link up or down the list
by changing a ranking number in the mysql database.
At first, my code seemed to work just fine untill i added a few more links and just moved them up and down at random to check that everything works as it should. at some point some links started to get the
same ranking number, which messed up the whole list.
The code is:
[code]if (isset($_POST['group_up']))
{
if ($_POST['upvalue']==1)
{
header( 'location:menu.php' );
}
else
{
$old = $_POST['upvalue'];
$up = $_POST['upvalue']-1;
$up_id = $_POST['group_id'];
$conn = mysql_connect("$db_host","$db_user","$db_pass");
if (!$conn) die ("Could not connect MySQL");
mysql_select_db($db_database,$conn) or die ("Could not open database");
$query = "UPDATE menu_groups SET ranking = $old WHERE ranking = $up;";
mysql_query($query) or die('Error, query failed');
$query = "UPDATE menu_groups SET ranking = $up WHERE group_id = $up_id;";
mysql_query($query) or die('Error, query failed');
}
}[/code]

And similar for the down button just it adds to the upvalue istead of subtract.
Annyone notice why this code bugs?
Also if you have suggestions on how to do this another way please tell me.

Thanks for your help
my table have these columns:

id, title and ranking

No two rows should have the same ranking number.

The ranking number is for the ordering when I shal output the results.
a low ranking means high placement in the output list.

so if my table conains

id | title | ranking
1 | Group1 | 2
2 | Group2 | 1

group2 with id 2 will come first.

What I want to do is to be able to change these ranking numbers in my admin page. this means that if I want to move an entry up the list, the ranking is lowered by one for this item. This also means that I have to make the entry with the previuos ranking the moved entry got, obtain the old ranking of the moved entry to ensure that no rows contains the same ranking number.

I hope that made sense..
This thread is more than a year old. Please don't revive it unless you have something important to add.

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue.