Z-Push Zimbra Backend / Bugs / #67 zimbra.php: Fix ZLog error if searchfolderid is not defined

#67 zimbra.php: Fix ZLog error if searchfolderid is not defined

Milestone: Release 67

Status: closed

Owner: nobody

Labels: None

Priority: 1

Updated: 2017-02-21

Created: 2017-02-07

Creator: Gregor Hoffleit

Private: No

In GetMailboxSearchResults(), the ZLog:Write for SearchFolderId is at the wrong place, which results in an error "Undefined variable: searchfolderid (8)", if searchfolderid is not defined.

This affects Release 66 and earlier.

The (trivial) fix is attached: Move the ZLog:Write inside the if clause (it is already that way in the similar code block in GetDocumentLibrarySearchResults()).

1 Attachments

bugfix.diff

Discussion

LiverpoolFCfan - 2017-02-09

Do you know how you generated a search with no folder specified?

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

LiverpoolFCfan - 2017-02-21

Managed to reproduce this with an iOS 10 device. Fixed it. Thanks. The fix will be in Release 67.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

LiverpoolFCfan - 2017-02-21

status: open --> closed

Group: unconfirmed --> Release 67
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

LiverpoolFCfan - 2017-02-21

Trivial issue. Closing ahead of fix being released

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

zimbra.php: Fix ZLog error if searchfolderid is not defined

Group

Searches

Help

#67 zimbra.php: Fix ZLog error if searchfolderid is not defined

Discussion