[1.0.0-beta1] IconDefinition methods interface

I think we already replaced getPath() with getSource() to be more generic.

Still 'content' is part of the renderable, so even if for now only svg extractor use it, the fact that we pass the optional content around make it generic enough I think, to discuss.

I think we already replaced getPath() with getSource() to be more generic.

Indeed, getSource() instead of getPath() because sources can be URL.

IconDefinition:getRenderable() must not hardcode content which is a data specific to SvgExtractor:

( 'content' => new FormattableMarkup($this->getContent(), []),)

Do we need to add an attribute in IconExtractor where the extractor can tell which specific data it is collecting:

#[IconExtractor(
  id: 'svg',
  label: new TranslatableMarkup('SVG'),
  description: new TranslatableMarkup('All files from one or many paths. Works for any file type.'),
  collected_data: [
    'content' => "string"
  ],
  forms: [
    'settings' => IconPackExtractorForm::class,
  ]
)]

And then use this information in IconDefinition or elsewhere.

Fixed in 📌 [Meta Beta release] Clean code, no default template, better preview Fixed