Bug fix: correct bug if updating a bookmark with the url of another existing bookmark.
git-svn-id: https://semanticscuttle.svn.sourceforge.net/svnroot/semanticscuttle/trunk@303 b3834d28-1941-0410-a4f8-b48e95affb8f
This commit is contained in:
parent
4386a63ec1
commit
d833e18b68
1 changed files with 10 additions and 1 deletions
|
@ -201,7 +201,7 @@ class BookmarkService {
|
|||
|
||||
function updateBookmark($bId, $address, $title, $description, $privateNote, $status, $categories, $date = NULL, $fromApi = false) {
|
||||
if (!is_numeric($bId))
|
||||
return false;
|
||||
return false;
|
||||
|
||||
// Get the client's IP address and the date; note that the date is in GMT.
|
||||
if (getenv('HTTP_CLIENT_IP'))
|
||||
|
@ -213,6 +213,15 @@ class BookmarkService {
|
|||
$ip = getenv('HTTP_X_FORWARDED_FOR');
|
||||
|
||||
$moddatetime = gmdate('Y-m-d H:i:s', time());
|
||||
|
||||
$address = $this->normalize($address);
|
||||
|
||||
//check if a new address ($address) doesn't already exist for another bookmark from the same user
|
||||
$bookmark = $this->getBookmark($bId);
|
||||
if($bookmark['bAddress'] != $address && $this->bookmarkExists($address, $bookmark['uId'])) {
|
||||
message_die(GENERAL_ERROR, 'Could not update bookmark (URL already existing = '.$address.')', '', __LINE__, __FILE__);
|
||||
return false;
|
||||
}
|
||||
|
||||
// Set up the SQL update statement and execute it.
|
||||
$updates = array('bModified' => $moddatetime, 'bTitle' => $title, 'bAddress' => $address, 'bDescription' => $description, 'bPrivateNote' => $privateNote, 'bStatus' => $status, 'bHash' => md5($address));
|
||||
|
|
Loading…
Reference in a new issue