unification test was unstable, too
git-svn-id: https://semanticscuttle.svn.sourceforge.net/svnroot/semanticscuttle/trunk@464 b3834d28-1941-0410-a4f8-b48e95affb8f
This commit is contained in:
parent
63f82a4101
commit
39db481506
2 changed files with 30 additions and 20 deletions
|
@ -479,7 +479,8 @@ class SemanticScuttle_Service_Bookmark extends SemanticScuttle_DbService
|
|||
* @param string $sortOrder One of the following values:
|
||||
* "date_asc", "date_desc",
|
||||
* "title_desc", "title_asc",
|
||||
* "url_desc", "url_asc"
|
||||
* "url_desc", "url_asc",
|
||||
* "voting_asc", "voting_desc"
|
||||
* @param boolean $watched True if only watched bookmarks
|
||||
* shall be returned (FIXME)
|
||||
* @param integer $startdate Filter for creation date.
|
||||
|
@ -574,23 +575,29 @@ class SemanticScuttle_Service_Bookmark extends SemanticScuttle_DbService
|
|||
}
|
||||
|
||||
switch($sortOrder) {
|
||||
case 'date_asc':
|
||||
$query_5.= ' ORDER BY B.bModified ASC ';
|
||||
break;
|
||||
case 'title_desc':
|
||||
$query_5.= ' ORDER BY B.bTitle DESC ';
|
||||
break;
|
||||
case 'title_asc':
|
||||
$query_5.= ' ORDER BY B.bTitle ASC ';
|
||||
break;
|
||||
case 'url_desc':
|
||||
$query_5.= ' ORDER BY B.bAddress DESC ';
|
||||
break;
|
||||
case 'url_asc':
|
||||
$query_5.= ' ORDER BY B.bAddress ASC ';
|
||||
break;
|
||||
default:
|
||||
$query_5.= ' ORDER BY B.bModified DESC ';
|
||||
case 'date_asc':
|
||||
$query_5 .= ' ORDER BY B.bModified ASC ';
|
||||
break;
|
||||
case 'title_desc':
|
||||
$query_5 .= ' ORDER BY B.bTitle DESC ';
|
||||
break;
|
||||
case 'title_asc':
|
||||
$query_5 .= ' ORDER BY B.bTitle ASC ';
|
||||
break;
|
||||
case 'voting_desc':
|
||||
$query_5 .= ' ORDER BY B.bVoting DESC ';
|
||||
break;
|
||||
case 'voting_asc':
|
||||
$query_5 .= ' ORDER BY B.bVoting ASC ';
|
||||
break;
|
||||
case 'url_desc':
|
||||
$query_5 .= ' ORDER BY B.bAddress DESC ';
|
||||
break;
|
||||
case 'url_asc':
|
||||
$query_5 .= ' ORDER BY B.bAddress ASC ';
|
||||
break;
|
||||
default:
|
||||
$query_5 .= ' ORDER BY B.bModified DESC ';
|
||||
}
|
||||
|
||||
// Handle the parts of the query that depend on any tags that are present.
|
||||
|
|
|
@ -103,15 +103,18 @@ class BookmarkTest extends TestBase
|
|||
{
|
||||
$bs = $this->bs;
|
||||
|
||||
$uid = $this->addUser();
|
||||
$uid2 = $this->addUser();
|
||||
|
||||
$bs->addBookmark(
|
||||
'http://site1.com', "title", "description", 'note',
|
||||
0, array('tag1'), null, false, false,
|
||||
1
|
||||
$uid
|
||||
);
|
||||
$bs->addBookmark(
|
||||
"http://site1.com", "title2", "description2", 'note',
|
||||
0, array('tag2'), null, false, false,
|
||||
2
|
||||
$uid2
|
||||
);
|
||||
|
||||
$bookmarks = $bs->getBookmarks();
|
||||
|
|
Loading…
Reference in a new issue