No subject

Hi!

We discovered some problems with kronolith's calendar view using the kolab
backend. Some of our events are not shown in day view or week view. The day
view sometimes also displays some wrong events that should occur about a we=
ek
later.

We tracked the issue down to listEvents() in kronolith/lib/Driver/kolab.php=
.
This function returned only events that start and end within the given
period. Event that e.g. start before and end within the period are not
returned. That causes some interesting results in different views of
kronolith. The attached patch fixes that issue. Would be nice if the patch
will be added to cvs.

Regards

 S=C3=B6nke Schwardt

----------
assignedto: wrobel
files: kronolith_calendar_fix.patch
messages: 11607
nosy: bernhard, thomas, wrobel
priority: bug
status: chatting
title: [horde/horde-kronolith] List all events within a period
topic: horde, patch
________________________________________________
Kolab issue tracker <kolab-issues at intevation.de>
<https://intevation.de/roundup/kolab/issue1907>
________________________________________________

--192.168.12.1.1020.10139.1187696556.851.2232
Content-Type: application/octet-stream
Content-Disposition: attachment;
    filename="kronolith_calendar_fix.patch"
Content-Transfer-Encoding: base64

LS0tIGtyb25vbGl0aC5vcmlnL2xpYi9Ecml2ZXIva29sYWIucGhwCTIwMDctMDgtMjAgMTM6Mzg6
MDYuMDAwMDAwMDAwICswMjAwCisrKyBrcm9ub2xpdGgvbGliL0RyaXZlci9rb2xhYi5waHAJMjAw
Ny0wOC0yMCAxMzo0MToxOC4wMDAwMDAwMDAgKzAyMDAKQEAgLTEwODQsMTAgKzEwODQsMjUgQEAK
ICAgICAgICAgICAgIGVjaG8gIlN0YXJ0OiAiLiRzdGFydERhdGUtPmNvbXBhcmVEYXRlKCRldmVu
dC0+c3RhcnQpLiI8YnI+XG4iOwogICAgICAgICAgICAgZWNobyAiRW5kOiAiLiRlbmREYXRlLT5j
b21wYXJlRGF0ZSgkZXZlbnQtPmVuZCkuIjxicj5cbiI7CiAqLwotICAgICAgICAgICAgLy8gY2hl
Y2sgbm9ybWFsIGV2ZW50IGRhdGUKLSAgICAgICAgICAgIGlmICgkc3RhcnREYXRlLT5jb21wYXJl
RGF0ZSgkZXZlbnQtPnN0YXJ0KSA8PSAwCi0gICAgICAgICAgICAgICAgJiYgJGVuZERhdGUtPmNv
bXBhcmVEYXRlKCRldmVudC0+ZW5kKSA+PSAtMSkKLSAgICAgICAgICAgIHsKKwkJCS8qCisJCQkg
KiBjaGVjayBpZiBldmVudCBwZXJpb2QgaW50ZXJzZWN0cyB3aXRoIGdpdmVuIHBlcmlvZCAoc3Rh
cnREYXRlLCBlbmREYXRlKQorCQkJICoKKwkJCSAqIFBvc3NpYmxlIGNhc2VzIG9mIGludGVyc2Vj
dGlvbiBvZiBwZXJpb2QgQSBhbmQgcGVyaW9kIEI6CisJCQkgKgorCQkJICogICAgICAgICAgICAg
ICAgICAgICB8LS0tLS0tUGVyaW9kIEEtLS0tLS18CisJCQkgKiBjYXNlMTogICAgfC0tQi0tfAor
CQkJICogY2FzZTI6ICAgICAgICAgfC0tLS1CLS0tLXwKKwkJCSAqIGNhc2UzOiAgICAgICAgICAg
ICAgICAgICB8LS0tLUItLS0tfAorCQkJICogY2FzZTQ6ICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICB8LS0tLUItLS0tfAorCQkJICogY2FzZTU6ICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICB8LS1CLS18CisJCQkgKiBjYXNlNjogICAgICAgICB8LS0tLS0tLS0tLS0tLS0t
Qi0tLS0tLS0tLS0tLS18CisJCQkgKgorCQkJICogY2FzZSAxIGFuZCA1IHNob3VsZCByZXR1cm4g
RkFMU0UgPT0+IG5vIGludGVyc2VjdGlvbgorCQkJICogY2FzZSAyLDMsNCBhbmQgNiBzaG91bGQg
cmV0dXJuIFRSVUUgPT0+IGludGVyc2VjdGlvbgorCQkJICoKKwkJCSAqLworCQkJaWYgKCEoKCRl
bmREYXRlLT5jb21wYXJlRGF0ZVRpbWUoJGV2ZW50LT5zdGFydCkgPCAwKSB8fCAoJHN0YXJ0RGF0
ZS0+Y29tcGFyZURhdGVUaW1lKCRldmVudC0+ZW5kKSA+IDApKSkKKwkJCXsKICAgICAgICAgICAg
ICAgICAka2VlcF9ldmVudCA9IHRydWU7CiAgICAgICAgICAgICB9CiAKQEAgLTEwOTUsOSArMTEx
MCw4IEBACiAgICAgICAgICAgICBpZiAoISRrZWVwX2V2ZW50ICYmICRldmVudC0+cmVjdXJzKCkp
IHsKICAgICAgICAgICAgICAgICAkbmV4dCA9ICRldmVudC0+cmVjdXJyZW5jZS0+bmV4dFJlY3Vy
cmVuY2UoJHN0YXJ0RGF0ZSk7CiAKLSAgICAgICAgICAgICAgICBpZiAoJG5leHQgIT09IGZhbHNl
ICYmICEkZXZlbnQtPnJlY3VycmVuY2UtPmhhc0V4Y2VwdGlvbigkbmV4dC0+eWVhciwgJG5leHQt
Pm1vbnRoLCAkbmV4dC0+bWRheSkKLSAgICAgICAgICAgICAgICAgICAgJiYgJHN0YXJ0RGF0ZS0+
Y29tcGFyZURhdGUoJG5leHQpIDw9IDAKLSAgICAgICAgICAgICAgICAgICAgJiYgJGVuZERhdGUt
PmNvbXBhcmVEYXRlKCRuZXh0KSA+PSAwKQorICAgICAgICAgICAgICAgIGlmICgkbmV4dCAhPT0g
ZmFsc2UgJiYgISRldmVudC0+cmVjdXJyZW5jZS0+aGFzRXhjZXB0aW9uKCRuZXh0LT55ZWFyLCAk
bmV4dC0+bW9udGgsICRuZXh0LT5tZGF5KSAmJgorCQkJCQkoISgoJGVuZERhdGUtPmNvbXBhcmVE
YXRlVGltZSgkZXZlbnQtPnN0YXJ0KSA8IDApIHx8ICgkc3RhcnREYXRlLT5jb21wYXJlRGF0ZVRp
bWUoJGV2ZW50LT5lbmQpID4gMCkpKSkKICAgICAgICAgICAgICAgICB7CiAgICAgICAgICAgICAg
ICAgICAgICRrZWVwX2V2ZW50ID0gdHJ1ZTsKICAgICAgICAgICAgICAgICB9Cg==

--192.168.12.1.1020.10139.1187696556.851.2232--