[Kde-pim] Review Request: Emit calendarLoaded() in CalendarResources:load()

Sergio Martins iamsergio at gmail.com
Fri Feb 13 23:58:04 GMT 2009


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/65/
-----------------------------------------------------------

Review request for KDE PIM.


Summary
-------

KOrganizer works very well with resource loading if it's the resource view doing the loading, but doesn't get notified if another component loads the resource.

If you start kontact on the summary view, and then switch to Calendar, you have to hit refresh to see the incidences (the first time only).

The problem is that CalendarResources::load() isn't emitting calendarLoaded() like specified in KCal::Calendar API.


Diffs
-----

  trunk/KDE/kdepim/korganizer/actionmanager.cpp 925222 
  trunk/KDE/kdepimlibs/kcal/calendarresources.cpp 924907 

Diff: http://reviewboard.kde.org/r/65/diff


Testing
-------

Played around with switching views and resource reloading in kontact.


Thanks,

Sergio

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/



More information about the kde-pim mailing list