Support config for fetcher and parser plugins

Created on 25 September 2023, about 1 year ago
Updated 27 September 2023, about 1 year ago

While I was working on πŸ› Prevent post-dated posts from staying at the top of aggregated feeds Needs review I discovered that the Aggregator's settings form supports sub-forms for fetcher and parser plugins, contrary to what I said in #33 of that issue. But this sent me down a development rabbit hole. The problem with this is that we don't support configuration for those plugins in the schema. This module's settings have been completely free-form, "just stick things wherever and expect them to work" up until now. It makes plugin development difficult because a person couldn't have stored things in aggregator.settings without throwing schema errors. If anyone tried to develop plugins before now, then they would have needed to implement their own separate configuration.

I was initially just going to solve the problem for parser plugins, but due to a quirk of SettingsForm->buildForm() I had to make the same changes for fetchers. Processor plugins really need some love too even though the form quirk doesn't apply to them. So I might as well fix them all. Patch to follow.

πŸ“Œ Task
Status

Needs review

Version

2.0

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States dcam

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Production build 0.71.5 2024