{"action":"created","issue":{"url":"https://api.github.com/repos/openchange/openchange/issues/274","labels_url":"https://api.github.com/repos/openchange/openchange/issues/274/labels{/name}","comments_url":"https://api.github.com/repos/openchange/openchange/issues/274/comments","events_url":"https://api.github.com/repos/openchange/openchange/issues/274/events","html_url":"https://github.com/openchange/openchange/pull/274","id":68156684,"number":274,"title":"[nspi] Do not raise invalid bookmark error in addresses container withou...","user":{"login":"javierag","id":2778038,"avatar_url":"https://avatars.githubusercontent.com/u/2778038?v=3","gravatar_id":"","url":"https://api.github.com/users/javierag","html_url":"https://github.com/javierag","followers_url":"https://api.github.com/users/javierag/followers","following_url":"https://api.github.com/users/javierag/following{/other_user}","gists_url":"https://api.github.com/users/javierag/gists{/gist_id}","starred_url":"https://api.github.com/users/javierag/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/javierag/subscriptions","organizations_url":"https://api.github.com/users/javierag/orgs","repos_url":"https://api.github.com/users/javierag/repos","events_url":"https://api.github.com/users/javierag/events{/privacy}","received_events_url":"https://api.github.com/users/javierag/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-04-13T17:55:03Z","updated_at":"2015-04-14T20:02:53Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/openchange/openchange/pulls/274","html_url":"https://github.com/openchange/openchange/pull/274","diff_url":"https://github.com/openchange/openchange/pull/274.diff","patch_url":"https://github.com/openchange/openchange/pull/274.patch"},"body":"...t purported search\r\n\r\nThe \"All address list\" container is meant to be only used as organizer and do not contains\r\nthe purportedSearch attribute.\r\nThis case was not take in account so it always gived a 'Invalid bookmark' error instead of\r\na empty list. This behaviour can be checked in Outlook selecting 'All address list' in the dropdown at\r\nthe 'Select names' dialog.\r\n\r\nTo take it in account the functions dcesrv_do_NspiQueryRows, emsabp_ab_container_enum and emsabp_ab_fetch_filter has been modified.\r\n\r\nThe functions dcesrv_NspiResolveNames and dcesrv_NspiResolveNamesW have been modified to take in account that emsabp_ab_fetch_filter can now return a NULL filter but they continue to return 'Invalid bookmark' if called on'All address list'. I have not changed them to return empty lists because I was not able to reproduce this situation with them. This two functions also need to be reviewed in the future."},"comment":{"url":"https://api.github.com/repos/openchange/openchange/issues/comments/93042551","html_url":"https://github.com/openchange/openchange/pull/274#issuecomment-93042551","issue_url":"https://api.github.com/repos/openchange/openchange/issues/274","id":93042551,"user":{"login":"javierag","id":2778038,"avatar_url":"https://avatars.githubusercontent.com/u/2778038?v=3","gravatar_id":"","url":"https://api.github.com/users/javierag","html_url":"https://github.com/javierag","followers_url":"https://api.github.com/users/javierag/followers","following_url":"https://api.github.com/users/javierag/following{/other_user}","gists_url":"https://api.github.com/users/javierag/gists{/gist_id}","starred_url":"https://api.github.com/users/javierag/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/javierag/subscriptions","organizations_url":"https://api.github.com/users/javierag/orgs","repos_url":"https://api.github.com/users/javierag/repos","events_url":"https://api.github.com/users/javierag/events{/privacy}","received_events_url":"https://api.github.com/users/javierag/received_events","type":"User","site_admin":false},"created_at":"2015-04-14T20:02:53Z","updated_at":"2015-04-14T20:02:53Z","body":"Not very relevant because the 'All address lists\" is really and separator and the user should not click there because it will always get a empty list. I do not undestand why outlook does not make it unclickable."}}