#111 backend not working with githib community version

[burton]

Hello!

When I try to use the Zimbra Backend with the community Version it cant connect.
(https://github.com/fmbiete/Z-Push-contrib)

---

17/04/2015 11:04:49 [25251] [DEBUG] [xxx@capalino.de] -------- Start
17/04/2015 11:04:49 [25251] [ INFO] [xxx@capalino.de] Version='SVN-trunk-r1935' method='OPTIONS' from='xx.xx.77.xx' cmd='' getUser='edda@capalino.de' devId='' devType=''
17/04/2015 11:04:49 [25251] [DEBUG] [xx@capalino.de] NoPostRequestException: Options request - code: 1 - file: /var/www/xx/xx/xx/web/subdomains/z-push/index.php:151
17/04/2015 11:04:49 [25251] [DEBUG] [xxx@capalino.de] Including backend file: '/var/www/clients/client4/web8/web/subdomains/z-push/backend/zimbra/zimbra.php'

---

Outlook2013

Outlook tells me that it cant find a Exchange Server

[burton]

The diffstate.php differs from the original one:

github community Version looks like that:

static public function RowCmp($a, $b) {
if (is_numeric($a["id"]) && is_numeric($b["id"])) {
return $a["id"] < $b["id"] ? 1 : -1;
}
else {
return strcmp($a["id"], $b["id"]) < 0 ? 1 : -1;
}
}

[LiverpoolFCfan]

Ticket moved from /p/zimbrabackend/bugs/53/

[LiverpoolFCfan]

I have moved this as it is not a bug in the backend.

The backend is designed to work with the official z-push release. I have no control over any changes the github version might have made that would break compatability.

And, as mentioned in the open discussion post, I have chosen to try to get z-push moving to include community input rather than further fragmenting the product by creating additional forks.

As regards the difference in diffstate.php - if you read the INSTALL note that comes with the zimbra backend you will find the reason for the difference - and why zimbra requires a change to that file.

[burton]

Hello!
What kind of logs do you need? The only thing i get is in the first post.

[LiverpoolFCfan]

You must/should be getting an error in your apache logs then. Do you have apache and php logs turned up?

[LiverpoolFCfan]

I have downloaded a ZIP of the latest version in that repository, and as suspected the version in the repository has broken the published z-push interface.

It has added a new abstract function definition to the diffbackend.php file upon which the zimbra backend is based. The zimbra backend does not implement this function - thus a fatal error is thrown.

    /**

     * Changes the 'star' flag of a message on disk. The $flags
     * parameter can only be '2' (starred) or '0' (unstarred). After a call to
     * SetStarFlag(), GetMessageList() should return the message with the
     * new 'flags' but should not modify the 'mod' parameter. If you do
     * change 'mod', simply setting the message to 'starred' on the mobile will trigger
     * a full resync of the item from the server.
     *
     * @param string        $folderid       id of the folder
     * @param string        $id             id of the message
     * @param int           $flags          star flag of the message
     * @param ContentParameters   $contentParameters
     *
     * @access public
     * @return boolean                      status of the operation
     * @throws StatusException              could throw specific SYNC_STATUS_* exceptions
     */
    public abstract function SetStarFlag($folderid, $id, $flags, $contentParameters);

I have already handled flags differently in the zimbra backend without breaking the interface, and changing the backend to support this method of doing so would require significant work. I am not prepared to expend that effort unless the z-push team adopts the change to the base product.

[LiverpoolFCfan]

If you run with debug logging on, hitting the url in a browser shows the error

( ! ) Fatal error: Class BackendZimbra contains 1 abstract method and must therefore be declared abstract or implement the remaining methods (BackendDiff::SetStarFlag) in C:\wamp2.5\www\z-push\Z-Push-contrib-master\backend\zimbra\zimbra.php on line 554

[burton]

Thank you, for your help. So I will wait and use the official Z-Push Repo. Hopefully Timbra will do something! Have a nice day!

[LiverpoolFCfan]

Timbra? You mean Zarafa?

[burton]

jep ;-)

[LiverpoolFCfan]

Closing out this ticket as it is not a backend issue - but rather a bad update to the github version that breaks compatibility with the base product.

[LiverpoolFCfan]

Just an FYI. I was in contact with Francisco who maintains the github version and explained that you had tried to use his version with the zimbra backend and that is crashes because of the new function he had added to the interface.

He said he is actually no longer using that function, and so has removed it.

You could try it again with that z-push version if you wish to do.