[horde] Finished importing horde FW_6 to packagist.org / composer / compatibility with newer PHP versions

Thu Dec 28 05:06:52 UTC 2023

Hello Brent,

Am 27/12/2023 um 22:44 schrieb Brent:
> Ok, thanks...that fixed it. I note that a few fixes that were discussed
> in this list aren't in included. You say you want a "PR", but what is
> that? Is there a mechanism for reporting issues? No problem to do that,
> I just don't know what a PR is, offhand.

I am fine with any kind of report. ML is OK, Wiki page is OK for now, 
Horde Ticket System is most preferred.

Tickets :: New Ticket (horde.org) <https://bugs.horde.org/ticket/create.php>

A Pull Request is a type of submitting a patch through Github together 
with a means to discuss and change the patch.
Those who send pull requests don't need to open a ticket or another 
means of communication, everything will happen right there.

> To wit, there's a type-o in the turba package. If you have an image
> for a contact, it'll trip up on it. There's a capital "S" instead of
> a lowercase one:
>
> <install dir>/vendor/horde/core/lib/Horde/Core/Ui/VarRenderer/Html.php
>
> Line 209....replace with the capital "S" after the id=
>
> Broken:
>
> $html .= sprintf('<input class="button" name="%s" id="%S" type="submit" value="%s" /> ',
>
> Fixed:
>
> $html .= sprintf('<input class="button" name="%s" id="%s" type="submit" value="%s" /> ',
>
>
> brent

That one confuses me. You say it's still broken and that your preferred 
/ fixed version has lowercase s

Core/lib/Horde/Core/Ui/VarRenderer/Html.php at FRAMEWORK_6_0 · 
horde/Core (github.com) 
<https://github.com/horde/Core/blob/FRAMEWORK_6_0/lib/Horde/Core/Ui/VarRenderer/Html.php#L209>

We committed that change in October 25.

Update Html.php · horde/Core at 7e1e507 (github.com) 
<https://github.com/horde/Core/commit/7e1e507b7d01e36f2e96a93d222fed9a5fb16ca1>

v3.0.0alpha15 has this patch, too.

Can you describe how you end up with the broken version when installing 
/ upgrading?

Regards

Ralf

>
> -----Original Message-----
>> From: "Lang, Ralf"<ralf.lang at gmail.com>
>> Date: Wed, 27 Dec 2023 08:12:42 +0100
>> To:horde at lists.horde.org,impuser at bitrealm.com
>> Subject: Re: [horde] Finished importing horde FW_6 to packagist.org /
>>   composer / compatibility with newer PHP versions
>> X-Spam-Status: No, No
>>
>> Hi Brent,
>>
>> thank you for reporting.
>>
>> I have fixed this now and added an install test for ingo's set of packages.
>>
>> Workflow runs · horde/bundle (github.com)
>> <https://github.com/horde/bundle/actions>
>>
>> This will probably improve as I go along
>>
>>
>> Am 26/12/2023 um 19:44 schrieb Brent:
>>>   I note a missing "horde/managesieve" component, required for ingo
>>> (rules) now?
>>>
>>> Happy Holidays!
>>>
>>> brent
>>>
>>> Quotingralf.lang at ralf-lang.de:
>>>
>>>> Hi Michael,
>>>>
>>>> I will do that and apply any missing / reported fixes. But no
>>>> timeline - I am with the family these days.Wish you all a merry
>>>> christmas and a happy new year.
>>>>
>>>> Am 23. Dezember 2023 12:26:25 MEZ schrieb Michael Schlueter
>>>> <michael at johalla.de>:
>>>>> Hello Ralf,
>>>>>
>>>>> What a great early Christmas gift. It works great for me :-)
>>>>>
>>>>> Would it be possible to create the horde/mapi package so I can install
>>>>> activesync?
>>>>>
>>>>> I had an issue with turba (Non-static method Horde_Form::getType()
>>>>> cannot be called statically) but a fix is already documented under:
>>>>> https://wiki.horde.org/Doc/Dev/H6/Developer/KnownIssues
>>>>>
>>>>> Also I can see the rampage warnung you can find on the KnowIssues
>>>>> page. This time I changed it to
>>>>> require_once dirname(__FILE__, 4) . '/vendor/autoload.php';
>>>>> Which seems to solve it.
>>>>>
>>>>> Thanks a lot and have a merry Christmas.
>>>>>
>>>>> Cheers
>>>>> Michael
>>>>>
>>>>> Zitat von Lang, Ralf<ralf.lang at gmail.com>:
>>>>>
>>>>>> Hello,
>>>>>>
>>>>>> Finally, all horde applications and libraries have been imported to
>>>>>> the global composer repository, packagist.org
>>>>>>
>>>>>> You can find the index here Packagist
>>>>>> <https://packagist.org/packages/horde/>
>>>>>>
>>>>>> Original code from the maintaina-com fork has been rebased on the
>>>>>> current master branch. I imported the original tags from maintaina.
>>>>>> These are alpha versions. I did not do anything about pear.
>>>>>> The only changes to master affect the composer.json file. I made it
>>>>>> compatible with current versions of composer and packagist but I
>>>>>> doubt anybody really used them in their previous state.
>>>>>>
>>>>>> The "master" branch is generally left alone for those who need to
>>>>>> run horde in its previous state based on PHP 7.2 - New development
>>>>>> targets the FRAMEWORK_6_0 branch
>>>>>>
>>>>>> Maintaina's FRAMEWORK_6_0 branch originally targeted PHP 7.4, 8.0
>>>>>> and 8.1 and is known to work on these versions within reasonable
>>>>>> limits of testing.
>>>>>>
>>>>>> Active upstream support for PHP 8.1 has ended some weeks ago.
>>>>>> Security fixes are available for the next 11 months.
>>>>>> I am currently working on full support for PHP 8.2 and 8.3.
>>>>>>
>>>>>> I just finished the importing work. While the unit tests for almost
>>>>>> all code has been ported to PHPUnit 9 and SHOULD work on the latest
>>>>>> PHP, the CI pipelines are currently broken.
>>>>>> This is a high priority currently.
>>>>>>
>>>>>> If you find any patch from master branch missing in FRAMEWORK_6_0,
>>>>>> please create a PR or simply comment if you cannot create the patch
>>>>>> yourself.
>>>>>>
>>>>>> Regards
>>>>>>
>>>>>> Ralf
>>>>>> -- 
>>>>>> Horde mailing list
>>>>>> Frequently Asked Questions:http://horde.org/faq/
>>>>>> To unsubscribe, mail:horde-unsubscribe at lists.horde.org
>>>>> -- 
>>>>> Horde mailing list
>>>>> Frequently Asked Questions:http://horde.org/faq/
>>>>> To unsubscribe, mail:horde-unsubscribe at lists.horde.org
>>>> -- 
>>>> Horde mailing list
>>>> Frequently Asked Questions:http://horde.org/faq/To  unsubscribe,
>>>> mail:horde-unsubscribe at lists.horde.org