From noreply at bugs.horde.org Mon Sep 13 13:29:00 2021 From: noreply at bugs.horde.org (noreply at bugs.horde.org) Date: Mon, 13 Sep 2021 13:29:00 +0000 Subject: [Tickets #13342] Re: Smartmobile view fails to load with PHP 5.3.3 In-Reply-To: References: Message-ID: <20210913132901.Horde.tj-5p7dsVgDuYL5Bb86NCRr@bugs.horde.org> DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED. Ticket URL: https://bugs.horde.org/ticket/13342 ------------------------------------------------------------------------------ Ticket | 13342 Updated By | fival18688 at shensufu.com Summary | Smartmobile view fails to load with PHP 5.3.3 Queue | Horde Framework Packages Version | Git master Type | Bug State | Not A Bug Priority | 1. Low Milestone | Patch | Owners | Michael Rubinsky ------------------------------------------------------------------------------ fival18688 at shensufu.com (2021-09-13 13:29) wrote: > After the update to Horde 5.2, the initial redirect from login.php > to the initial application fails when using the smartmobile view. > > I end up in an infinite redirection loop. My observations so far: > * the initial page is /imp/smartmobile.php#mailbox?mbox=SU5CT1g (correct) > * the call to Horde::url() returns > https://host:port/#mailbox?mbox=SU5CT1g (wrong) > > I assume the error is due to changes in parse_url(), because: > on the production system (PHP Version 5.3.3-7+squeeze19) > var_dump(parse_url('/imp/smartmobile.php#mailbox?mbox=SU5CT1g')); > array(1) { > 'fragment' => > string(20) "mailbox?mbox=SU5CT1g" > } > > and with a more recent version (php -v: PHP 5.5.9-1ubuntu4 (cli) > (built: Apr 9 2014 17:11:57)) > > var_dump(parse_url('/imp/smartmobile.php#mailbox?mbox=SU5CT1g')); > array(2) { > ["path"]=> > string(20) "/imp/smartmobile.php" > ["fragment"]=> > string(20) "mailbox?mbox=SU5CT1g" > } > > What would be the right way to deal with this? From noreply at bugs.horde.org Mon Sep 13 15:12:20 2021 From: noreply at bugs.horde.org (noreply at bugs.horde.org) Date: Mon, 13 Sep 2021 15:12:20 +0000 Subject: [Tickets #15075] Copy-Paste-Error in Horde Timezone Parsing Message-ID: BITTE NICHT AUF DIESE NACHRICHT ANTWORTEN. NACHRICHTEN AN DIESE E-MAIL-ADRESSE WERDEN NICHT GELESEN. Ticket-URL: https://bugs.horde.org/ticket/15075 ------------------------------------------------------------------------------ Ticket | 15075 Erstellt Von | wahnes at uni-koeln.de Zusammenfassung | Copy-Paste-Error in Horde Timezone Parsing Warteschlange | Kronolith Version | FRAMEWORK_5_2 Typ | Bug Status | Unconfirmed Priorität | 2. Medium Milestone | Patch | 1 Zuständige | ------------------------------------------------------------------------------ wahnes at uni-koeln.de (2021-09-13 15:12) hat geschrieben: There is a bug in both the current Git master and the stable (5.2) branch concerning the parsing of Timezone rules. The error is on line 276 of /usr/share/pear/Horde/Timezone/Rule.php (when using a PEAR install). It seems that the the "elseif" block was inserted using copy/paste from the previous block and that the "smaller or equal" condition was adjusted only on the "if" line, but not on the "explode" line. So this causes trouble with calendar entries that actually make use of repetition rules that are valid only for certain days of a month. I hope that the attached patch fixes the issue completely. I haven't done a very thorough check, though. wahnes at uni-koeln.de (2021-09-13 15:12) hat hochgeladen: horde-timezone-rule-for-kronolith.patch https://bugs.horde.org/h/services/download/?app=whups&actionID=download_file&file=horde-timezone-rule-for-kronolith.patch&ticket=15075&fn=%2Fhorde-timezone-rule-for-kronolith.patch From noreply at bugs.horde.org Tue Sep 14 00:33:48 2021 From: noreply at bugs.horde.org (noreply at bugs.horde.org) Date: Tue, 14 Sep 2021 00:33:48 +0000 Subject: [Tickets #15075] Re: Copy-Paste-Error in Horde Timezone Parsing In-Reply-To: References: Message-ID: <20210914003348.Horde.AYbyzu_drhpYl9ukf4Fcxus@bugs.horde.org> BITTE NICHT AUF DIESE NACHRICHT ANTWORTEN. NACHRICHTEN AN DIESE E-MAIL-ADRESSE WERDEN NICHT GELESEN. Ticket-URL: https://bugs.horde.org/ticket/15075 ------------------------------------------------------------------------------ Ticket | 15075 Aktualisiert Von | wahnes at uni-koeln.de Zusammenfassung | Copy-Paste-Error in Horde Timezone Parsing Warteschlange | Kronolith Version | FRAMEWORK_5_2 Typ | Bug Status | Unconfirmed Priorität | 2. Medium Milestone | Patch | 1 Zuständige | +Neuer Anhang | horde-timezone-rule-for-kronolith.patch ------------------------------------------------------------------------------ wahnes at uni-koeln.de (2021-09-14 00:33) hat geschrieben: > The error is on line 276 of /usr/share/pear/Horde/Timezone/Rule.php > (when using a PEAR install). It seems that the the "elseif" block > was inserted using copy/paste from the previous block It looks like there is a similar issue in the same file, but a bit before, around line 214. There, too, the '<=' case does not seem to be handled properly, whereas the '>=' case is. So here is a revised patch to take care of that part as well. wahnes at uni-koeln.de (2021-09-14 00:33) hat hochgeladen: horde-timezone-rule-for-kronolith[1].patch https://bugs.horde.org/h/services/download/?app=whups&actionID=download_file&file=horde-timezone-rule-for-kronolith%5B1%5D.patch&ticket=15075&fn=%2Fhorde-timezone-rule-for-kronolith%5B1%5D.patch From noreply at bugs.horde.org Wed Sep 15 13:54:41 2021 From: noreply at bugs.horde.org (noreply at bugs.horde.org) Date: Wed, 15 Sep 2021 13:54:41 +0000 Subject: [Tickets #14903] Re: Share Mailboxes (ACL editor) should indicate insufficient rights In-Reply-To: References: Message-ID: <20210915135441.Horde.93Zeqd3Qq8ZtfbLuQObbziZ@bugs.horde.org> BITTE NICHT AUF DIESE NACHRICHT ANTWORTEN. NACHRICHTEN AN DIESE E-MAIL-ADRESSE WERDEN NICHT GELESEN. Ticket-URL: https://bugs.horde.org/ticket/14903 ------------------------------------------------------------------------------ Ticket | 14903 Aktualisiert Von | wahnes at uni-koeln.de Zusammenfassung | Share Mailboxes (ACL editor) should indicate | insufficient rights Warteschlange | IMP Version | 6.2.23 Typ | Enhancement Status | New Priorität | 1. Low Milestone | Patch | Zuständige | ------------------------------------------------------------------------------ omniflux at omniflux.com (2019-03-01 04:16) hat geschrieben: In Mail/Preferences/Share Mailboxes, if a user does not have the administer permission for the selected mailbox, there should be some indication such as a notice, or options changed to read only. Also, instead of incorrectly populating the checkboxes, when GETACL fails, it could fall back to MYRIGHTS to populate the current users ACL checkboxes. omniflux at omniflux.com (2019-03-01 04:16) hat hochgeladen: ACL.png https://bugs.horde.org/h/services/download/?app=whups&actionID=download_file&file=ACL.png&ticket=14903&fn=%2FACL.png From noreply at bugs.horde.org Wed Sep 15 13:55:09 2021 From: noreply at bugs.horde.org (noreply at bugs.horde.org) Date: Wed, 15 Sep 2021 13:55:09 +0000 Subject: [Tickets #14870] Re: Changing mailbox ACLs recursively In-Reply-To: References: Message-ID: <20210915135509.Horde.P7ELQa5lGYQTgI9sD3fmuLF@bugs.horde.org> BITTE NICHT AUF DIESE NACHRICHT ANTWORTEN. NACHRICHTEN AN DIESE E-MAIL-ADRESSE WERDEN NICHT GELESEN. Ticket-URL: https://bugs.horde.org/ticket/14870 ------------------------------------------------------------------------------ Ticket | 14870 Aktualisiert Von | wahnes at uni-koeln.de Zusammenfassung | Changing mailbox ACLs recursively Warteschlange | IMP Version | 6.2.22 Typ | Enhancement Status | New Priorität | 2. Medium Milestone | Patch | Zuständige | ------------------------------------------------------------------------------ frank.richter at hrz.tu-chemnitz.de (2018-10-30 14:07) hat geschrieben: Hi, it would be a great enhancement to have a possibility to change an ACL for a mailbox recursively for all subfolders, too. Especially for shared folders. Thanks, Frank