Files
ttrss-plugin-af-fulltext/patches/README.md

34 lines
1.5 KiB
Markdown
Raw Normal View History

# Local patches to the vendored dependencies
`vendor/` is committed (tt-rss plugins ship their dependencies; there is no
composer in the app container), so upstream fixes we cannot wait for are applied
here and recorded below.
## fivefilters/readability.php
- `src/Nodes/NodeTrait.php:371``callable $filterFn = null` → `?callable
$filterFn = null`. Implicitly nullable parameters are deprecated as of PHP 8.4,
and the tt-rss app image is on PHP 8.5, so without this every extraction emits a
deprecation notice.
## composer autoloader class name
`vendor/` was seeded by copying `tt-rss-plugin-af-readability`'s tree, so both
plugins shipped the same generated bootstrap classes —
`ComposerAutoloaderInitb44cc79a…` and `ComposerStaticInitb44cc79a…`.
tt-rss's Preferences page (`Pref_Prefs::getPluginsList()``PluginHost::load_all()`)
loads **every** plugin on disk, enabled or not. The second `require vendor/autoload.php`
then redeclared those classes and PHP fataled, which tt-rss reports as:
> Plugin af_readability has caused a PHP fatal error so it won't be loaded again
> in this session.
— naming whichever plugin lost the race, not the one at fault.
The hash in `vendor/autoload.php`, `vendor/composer/autoload_real.php` and
`vendor/composer/autoload_static.php` is therefore rewritten to
`affulltext0000000000000000000001`. Regenerating `vendor/` with composer would
also fix it (composer derives the hash per-install); if you ever do, keep the
name distinct from any other tt-rss plugin's.