Re: [Evolution] WebCal import fails

2015-05-04 Thread Milan Crha
On Sun, 2015-05-03 at 22:05 +0200, Jay Strict wrote:
> == snip ==
> 
> (evolution-calendar-factory-subprocess:1824): libecalbackendhttp
> -WARNING
> **:  The component does not have the  mandatory property UID
> 
> == snap ==

Hi,
that error may explain it. Use 'wget' to get the .ics file from the
web and check its content whether it has the UID or not (just search
for 'UID'). Based on the error messages there are not set those, at
least not on some (if not all) of the components. Let the author know
to generate the .ics file in a correct format [1] (I hope they do not
claim too old version of the standard).
Bye,
Milan

[1] http://tools.ietf.org/html/rfc2445#section-4.8.4.7
___
evolution-list mailing list
evolution-list@gnome.org
To change your list options or unsubscribe, visit ...
https://mail.gnome.org/mailman/listinfo/evolution-list


Re: [Evolution] WebCal import fails

2015-05-04 Thread Jay Strict
On 2015-05-04 at 10:15, Milan Crha wrote:
> On Sun, 2015-05-03 at 22:05 +0200, Jay Strict wrote:
>> == snip ==
>>
>> (evolution-calendar-factory-subprocess:1824): libecalbackendhttp
>> -WARNING
>> **:  The component does not have the  mandatory property UID
>>
>> == snap ==
> 
> Hi,
> that error may explain it. Use 'wget' to get the .ics file from the
> web and check its content whether it has the UID or not (just search
> for 'UID'). Based on the error messages there are not set those, at
> least not on some (if not all) of the components. Let the author know
> to generate the .ics file in a correct format [1] (I hope they do not
> claim too old version of the standard).
> Bye,
> Milan
> 
> [1] http://tools.ietf.org/html/rfc2445#section-4.8.4.7

Yes you are absolutely correct. The .ics file does *not* contain any UID
field, so it is corrupt (w.r.t the standard).


Buut, ... ;)

the .ics file is displayed on e.g. Apple iPhones without errors.
Furthermore the .ics file is provided by an institution which is not
technically adept. These people don't care about Evolution, they only
care whether the calendar can be displayed on their phones.
Trying to convince them to fix their website feels like fighting a
losing battle.

The Result:
1) I cannot use Evolution with this website so I am driven away from
Evolution.
2) Everybody who wants to try Evolution is put off if she sees that
Evolution can't handle these files (they get the impression that the
software on iPhone is superior to Evolution, even though the problem is
in fact the faulty .ics file)


Would it not be better to
1) display a warning that the .ics file is damaged and
2) try to display the VEVENTS, nevertheless?


But maybe it is too much of an effort...?




[On a side note, let me state that I feel it is a bug in Evolution that
Evolution does display the error messages *only* in the log files.
Regarding the GUI one gets the impression that the .ics file was
imported just fine.
Would you agree that this is a bug? Should I report it to the bug tracker?]


Kind regards,
Jay
___
evolution-list mailing list
evolution-list@gnome.org
To change your list options or unsubscribe, visit ...
https://mail.gnome.org/mailman/listinfo/evolution-list


Re: [Evolution] WebCal import fails

2015-05-04 Thread Milan Crha
On Mon, 2015-05-04 at 14:30 +0200, Jay Strict wrote:
> But maybe it is too much of an effort...?

Hi,
not necessarily too much. The problem is that evolution uses the UID
(and eventually RID - recurrence ID) as an identifier of the event,
thus it can operate with it in the UI, and even with the local cache.

Is the calendar public, thus anyone can use it, or it's a semi-private
calendar? Being it a public calendar maybe someone can look on it. All
there matters is the version of the .ics file. The RFC I gave a link
to describes version 2.0. Earlier versions didn't have the UID
mandatory. Maybe a change in the On The Web calendar could be done to
add artificial UID properties for older versions of the .ics files.

> [On a side note, let me state that I feel it is a bug in Evolution 
> that Evolution does display the error messages *only* in the log 
> files. Regarding the GUI one gets the impression that the .ics file 
> was imported just fine. Would you agree that this is a bug? Should I 
> report it to the bug tracker?]

It depends on the point of view. Say you've 100 events in the .ics
file. Each being affected by a different issue, some with missing UID,
some with other issues. How would you report such issue to the user?
Collecting all the errors and sending them all at once, cluttering the
UI? The other thing is that with the On The Web calendars you usually
do not have any way to influence the content, as in your case. I know
it's no strong argument, I'm just thinking aloud. I agree that there
could be a notice that not all events (tasks or memos) were shown from
the .ics file (this answers one of my above questions too).
Bye,
Milan

___
evolution-list mailing list
evolution-list@gnome.org
To change your list options or unsubscribe, visit ...
https://mail.gnome.org/mailman/listinfo/evolution-list


Re: [Evolution] WebCal import fails

2015-05-04 Thread Jay Strict
On 2015-05-04 at 14:47, Milan Crha wrote:
> Is the calendar public, thus anyone can use it, or it's a semi-private
> calendar? Being it a public calendar maybe someone can look on it. All
> there matters is the version of the .ics file. The RFC I gave a link
> to describes version 2.0. Earlier versions didn't have the UID
> mandatory. Maybe a change in the On The Web calendar could be done to
> add artificial UID properties for older versions of the .ics files.

The first 3 lines of the .ics file are:


BEGIN:VCALENDAR
VERSION:2.0
PRODID:-//dhtmlXScheduler//NONSGML v2.2//EN


So I assume it is an iCal v2.0 file with malformed entries.
The calendar is semi-private, but I can send the entire file in a
private email if that matters to you.

> 
>> [On a side note, let me state that I feel it is a bug in Evolution 
>> that Evolution does display the error messages *only* in the log 
>> files. Regarding the GUI one gets the impression that the .ics file 
>> was imported just fine. Would you agree that this is a bug? Should I 
>> report it to the bug tracker?]
> 
> It depends on the point of view. Say you've 100 events in the .ics
> file. Each being affected by a different issue, some with missing UID,
> some with other issues. How would you report such issue to the user?
> Collecting all the errors and sending them all at once, cluttering the
> UI? The other thing is that with the On The Web calendars you usually
> do not have any way to influence the content, as in your case. I know
> it's no strong argument, I'm just thinking aloud. I agree that there
> could be a notice that not all events (tasks or memos) were shown from
> the .ics file (this answers one of my above questions too).

Cluttering the UI should certainly be avoided.
I would display an icon
http://commons.wikimedia.org/wiki/File:Gnome-dialog-warning.svg next to
the calendar in in the calendar list.
(Like the one with the two arrows (left-right) which is currently
displayed for all my calendars and for which I don't know the meaning.)
Clicking or hovering over the icon should then display a message like
"Malformed iCal file. This calendar may be displayed incompletely.
Please see logfiles for a detailed description."


Kind regards,
Jay
___
evolution-list mailing list
evolution-list@gnome.org
To change your list options or unsubscribe, visit ...
https://mail.gnome.org/mailman/listinfo/evolution-list


Re: [Evolution] WebCal import fails

2015-05-04 Thread Milan Crha
On Tue, 2015-05-05 at 00:07 +0200, Jay Strict wrote:
> The first 3 lines of the .ics file are:
> 
> 
> BEGIN:VCALENDAR
> VERSION:2.0
> PRODID:-//dhtmlXScheduler//NONSGML v2.2//EN
> 
> 
> So I assume it is an iCal v2.0 file with malformed entries.

Hi,
that's correct, they claim to be version 2.0, but the content is
malformed.

> The calendar is semi-private, but I can send the entire file in a
> private email if that matters to you.

Interestingly, saving the file and adding it as a custom On This
Computer calendar shows the events (with a newly added UIDs by the
backend).

> (Like the one with the two arrows (left-right) which is currently
> displayed for all my calendars and for which I don't know the 
> meaning.)

It's the connection state of the calendar, whether it's connected
and/or reachable. I agree that the change to the two-arrows icon is
confusing for me too.

> Clicking or hovering over the icon should then display a message like
> "Malformed iCal file. This calendar may be displayed incompletely.
> Please see logfiles for a detailed description."

Anything like that may require API changes.

As the local backend can cope with missing UIDs, I guess the read-only
On The Web backend can safely add missing UIDs as well.
Bye,
Milan

___
evolution-list mailing list
evolution-list@gnome.org
To change your list options or unsubscribe, visit ...
https://mail.gnome.org/mailman/listinfo/evolution-list


Re: [Evolution] WebCal import fails

2015-05-04 Thread Milan Crha
On Tue, 2015-05-05 at 07:50 +0200, Milan Crha wrote:
> As the local backend can cope with missing UIDs, I guess the
> read-only On The Web backend can safely add missing UIDs as well.

Hi,
I did so, for 3.17.2+ [1] and 3.16.2+ [2]. It requires to cleanup
~/.cache/evolution/calendar//
and restart evolution-calendar-factory, thus it'll pick the changes
and will fix the calendar content on the fly.

Still, the correct solution is to fix the calendar generator to
produce also UIDs for the components.
Bye,
Milan

[1] https://git.gnome.org/browse/evolution-data-server/commit/?id=513a9e8
[2] https://git.gnome.org/browse/evolution-data-server/commit/?id=261091c
___
evolution-list mailing list
evolution-list@gnome.org
To change your list options or unsubscribe, visit ...
https://mail.gnome.org/mailman/listinfo/evolution-list


Re: [Evolution] WebCal import fails

2015-05-05 Thread Jay Strict
On 2015-05-05 at 08:12, Milan Crha wrote:
> On Tue, 2015-05-05 at 07:50 +0200, Milan Crha wrote:
>> As the local backend can cope with missing UIDs, I guess the
>> read-only On The Web backend can safely add missing UIDs as well.
> 
> Hi,
> I did so, for 3.17.2+ [1] and 3.16.2+ [2]. It requires to cleanup
> ~/.cache/evolution/calendar//
> and restart evolution-calendar-factory, thus it'll pick the changes
> and will fix the calendar content on the fly.


Thank you for your time and your work. I appreciate it.

Jay
___
evolution-list mailing list
evolution-list@gnome.org
To change your list options or unsubscribe, visit ...
https://mail.gnome.org/mailman/listinfo/evolution-list