Two problems with php and mysql - lost and frustrated

sandstone

Question 1: Ok, here goes. I am building a music related website that allows artist to upload original music. I'm using open source software including xoops and several modules. One module includes an option for users to rate files (rate from 1 to 10) that have been uploaded. The problem is when a user rates a file then their rating is passed on to the database, but instead of adding this rating to the previous ratings it is replacing the previous rating and inserting the new rating. Ultimately, the database only has the rating from the most recent user who has rated. I need it to be adding all rating together.

Question 2: The module I'm using works great for uploads, but, the author did not include a way for files to be edited/deleted by the user. I'm lost and have no idea how to implement a user-side code to allow editing/deleting of files that are in the database.

Any help, simple points in the right direction, answers, assistance, just plain help is appriciated!!

Thanks
Sandy

Jack_McSlay

1 - this is a simple average problem. have the database store how many times the item has been rated and the sum of all ratings.
example for rating:

$rating = 5;
$musicid = 1;

mysql_query('UPDATE `musics` SET `points` = (`points`+'.$rating.'), `numRatings`=(`numRatings`+1) WHERE `musicId`='.$musicid);

example for getting the rating:

$musicid = 1;

$r = mysql_query('SELECT * FROM `musics` WHERE `musicId`='.$musicid);
$data = mysql_fetch_assoc($r);
$rating = $data['points'] / $data['numRatings']; //edit: it's a division, not a sum, doh

2 - why would one want to edit music from a website? anyway, if you want to delete, then I need to see how is the database structured, because some database data might have to be updated/deleted in order to prevent dead links

peytonrm

Editing binary files is beyond the scope of php. You would probably need some command line program that you could call with fpassthru(), but like the above poster said, I don't see the need.

To delete files you would just need to call unlink() and give it the filename to delete.

sandstone

Thanks to both for your help. I will give the code a try. Wish me luck, I have a feeling I'm going to need it.

I will also check out unlink(). The need is for users who have uploaded songs to be able to delete the songs they've uploaded. In example: maybe they have went into a studio and recorded a better version of the same song. Then they would want to delete the song from our website and upload the newer version.

Thanks again for your help. I'll let you know how it turns out

Sandy

benracer

http://uk.php.net/delete

sandstone

I need a crash course on php. I'm working with code that has already been written and I just need to alter it. Here's what I've discovered:

The user rates a song - the rated info is stored in a DB called 'votedata' which has fields for 'lid' (which is the songs ID#) and 'rating' (which holds the actual rating number from the last rater).

There's another DB called 'files' that has fields for all file info. One field is 'xfid' (which holds the song ID#) and one field is 'rating' (which holds the current rating for the song).

What is currently happening is the code is telling the DB to add a new row in DB 'votedata' and it is placing all info from the current rating into this new row. basically, the code is then copying the 'rating' data to the 'rating' field in the DB 'files'.

I need the code to take the data from the DB 'votedata' in the filed 'rating' and ADD IT TO the DB 'files' in the field 'rating', and to leave the sum in that same field.

This should be easy right!

The code I'm working with is open source from a xoops module called Debaser. the whole purpose for this is to set up a user voted chart of top rated songs.

Here's the section of code that I think I should be working on changing.

// All is well. Add to Line Item Rate to DB.
$newid = $xoopsDB->genId($xoopsDB->prefix('debaservotedata'). "ratingid_seq");
$datetime = time();
$xoopsDB->query("INSERT INTO ".$xoopsDB->prefix('debaservotedata')." (ratingid, lid, ratinguser, rating, ratinghostname, ratingtimestamp) VALUES ($newid, $lid, $ratinguser, $rating, '$ip', $datetime)" );
// All is well. Calculate Score & Add to Summary (for quick retrieval & sorting) to DB.
updatedebaserrating($lid);
$ratemessage = MD_DEBASER_VOTEAPPRE . "<br />" . sprintf(_MD_DEBASER_THANKYOU, $xoopsConfig['sitename']);
redirect_header("singlefile.php?id=$lid", 1, $ratemessage);
exit();

I know what I need to do, I just can't figure out how to do it. Can you help? Is there a "PHP and MySQL for Dummies" website out there somewhere?

My site is www.brimstonerecords.com if anyone is currious........🙂

sandstone

Ok, here's what I did:

// All is well. Add to Line Item Rate to DB.
$newid = $xoopsDB->genId($xoopsDB->prefix('debaservotedata'). "ratingid_seq");
$datetime = time();
$xoopsDB->query("INSERT INTO ".$xoopsDB->prefix('debaservotedata')." (ratingid, lid, ratinguser, rating, ratinghostname, ratingtimestamp) VALUES ($newid, $lid, $ratinguser, $rating, '$ip', $datetime)" );
//my coding
$result = $xoopsDB->query("SELECT rating FROM " .$xoopsDB->prefix('debaservotedata') " WHERE lid=$lid");
$xoopsDB->query('UPDATE 'rating' FROM " .$xoopsDB->prefix('debaser_files'). " SET 'rating' = ('.$result'+'.$rating.') "WHERE 'lid = $lid");
// All is well. Calculate Score & Add to Summary (for quick retrieval & sorting) to DB.
updatedebaserrating($lid);
$ratemessage = MD_DEBASER_VOTEAPPRE . "<br />" . sprintf(_MD_DEBASER_THANKYOU, $xoopsConfig['sitename']);
redirect_header("singlefile.php?id=$lid", 1, $ratemessage);
exit();

I get this error:

Parse error: syntax error, unexpected '"' in /a/directory/somewhere/on/my/websites/page/thefile.php on line 88

Any idea's? Lost again!!!!

Sandy

sandstone

//my coding
line 88 $result = $xoopsDB->query("SELECT rating FROM " .$xoopsDB- >prefix('debaservotedata') " WHERE lid=$lid");
line 89 $xoopsDB->query('UPDATE 'rating' FROM " .$xoopsDB->prefix('debaser_files'). " SET 'rating' = ('.$result'+'.$rating.') "WHERE 'lid = $lid");