dokuwiki_plugin_wrap icon indicating copy to clipboard operation
dokuwiki_plugin_wrap copied to clipboard

Fix deprecation warnings

Open fiwswe opened this issue 3 years ago • 2 comments

Fixes #242

Works under Hogfather and Igor.

fiwswe avatar Sep 07 '22 11:09 fiwswe

This will break the older Greebo version, because _addCall was refactored in Hogfather. For most plugins I would merge it already, but the Wrap plugin is one of the most used plugins. So I expect therefore relative more not yet updated installations are using it. @selfthinker do you prefer longer backward compatibility, more than two releases?

Klap-in avatar Sep 07 '22 20:09 Klap-in

I don't have any idea which versions are in actual use. But my 2¢ are: Hogfather is still important, while everything gets fixed and updated for Igor and PHP 8. Greebo not so much.

Looking at it from a different perspective: Hogfather was released more than 2 years ago. Dropping support for older systems should be acceptable.

But if required we could adjust the fix to differentiate between DW versions. Ugly and it opens a huge can of worms IMHO.

Plugin and Theme metadata and distribution lacks a mechanism to define DW compatibility. Or a mechanism to offer different versions for different DW versions. So when the DW API changes, there tend to be issues. Not sure if adding such a mechanism would be worthwhile. Something like that has the potential to add a lot of complexity for something not very important.

fiwswe avatar Sep 07 '22 21:09 fiwswe

I would agree Hogfather compatibility is enough. People who do not upgrade their wiki usually do not upgrade their plugins either ;-)

@selfthinker can we get this merged?

splitbrain avatar Oct 25 '22 08:10 splitbrain