After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 670944 - Option to handle out of office without date range
Option to handle out of office without date range
Status: RESOLVED FIXED
Product: evolution-ews
Classification: Other
Component: Mail
3.3.x
Other Linux
: Normal enhancement
: ---
Assigned To: Evolution EWS maintainer(s)
Evolution EWS maintainer(s)
: 670935 670936 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2012-02-28 08:07 UTC by Akhil Laddha
Modified: 2012-04-06 13:03 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
screen shot of out of office in owa (24.68 KB, image/png)
2012-02-28 08:07 UTC, Akhil Laddha
  Details
Evolution EWS patch (10.14 KB, patch)
2012-03-01 08:42 UTC, Punit Jain
committed Details | Review

Description Akhil Laddha 2012-02-28 08:07:43 UTC
Created attachment 208553 [details]
screen shot of out of office in owa

Evolution 3.3.91

OWA or Outlook supports out of office in two ways
1) Set out of office for given time period
2) Set out of office without time period

EWS doesn't support option 2. That will create problem when user sets out of office in owa because evolution will pick random dates and will stop sending out of office messages after that random period.
Comment 1 Punit Jain 2012-03-01 08:42:30 UTC
Created attachment 208773 [details] [review]
Evolution EWS patch

Thanks for bug report.
Add a check-box widget same as in owa/outlook to specify if user want to set particular duration or not.

This patch also include solution to following bugs too:
Show a warning if date is in past bug 670935.
Show Out of Office in bold bug 670936.

Thanks!
Comment 2 Punit Jain 2012-03-01 08:44:36 UTC
*** Bug 670936 has been marked as a duplicate of this bug. ***
Comment 3 Punit Jain 2012-03-01 08:45:11 UTC
*** Bug 670935 has been marked as a duplicate of this bug. ***
Comment 4 Chenthill P 2012-03-07 04:19:10 UTC
Review of attachment 208773 [details] [review]:

The rest of the UI changes looks fine.

::: src/server/e-ews-connection.c
@@ +675,3 @@
+		start = html_text;
+		end = html_text + haystack_len;
+	}

Is there a better way to parse html text ? :)
Comment 5 Punit Jain 2012-03-26 11:33:04 UTC
I looked for and I couldn't find better way for parsing the only relevant information because we are only interested in Message text and nothing else.
I looked in evolution-exchange too and it also uses same approach.

Thanks!
Comment 6 Punit Jain 2012-04-06 13:03:16 UTC
Review of attachment 208773 [details] [review]:

Patch has been committed to master branch.
http://git.gnome.org/browse/evolution-ews/commit/?id=b0ac43f31f8ec2a207d6e6d1715848864364de54

Note: update the parsing method later if we can find a good method.