[dev] [PATCH] trean: return status in addBookmark

Jan Schneider jan at horde.org
Fri May 14 02:02:13 PDT 2004


Zitat von Robert Trace <maillist-sender-94b1d4 at farcaster.org>:

> Trean doesn't appear to return any status when trying to add a bookmark.
>   If the backend fails to add the bookmark, the status is silently
> dropped and it appears to the calling routine as though the add
> succeeded when it most obviously didn't.  (Most obvious because my huge
> import of bookmarks missed several when the category filled up.)
>
> The attached patch corrects this by returning the status from the
> backend add operation.

Simplified and committed, thanks. I also added checks to data.php and
edit.php.

Jan.

--
Do you need professional PHP or Horde consulting?
http://horde.org/consulting/


More information about the dev mailing list