Move library discovery into Core\Asset\Discovery namespace and add 'asset.' prefix to service IDs

Created on 4 June 2014, over 10 years ago
Updated 20 August 2023, over 1 year ago

Problem/Motivation

From #2223143: Consolidate library extension caches into a single cache entry :

It would also be great to create a separate follow-up issue to (1) rename the service IDs into 'asset.library_discovery*' and optionally (2) move the classes into a \Asset\Discovery subnamespace and remove "Discovery" from the class names.

Proposed resolution

Discuss and implement this in some form.

Remaining tasks

User interface changes

None

API changes

Only if there are people extending/doing stuff with library services.

📌 Task
Status

Closed: won't fix

Version

9.5

Component
Asset library 

Last updated about 2 hours ago

No maintainer
Created by

🇬🇧United Kingdom damiankloip

Live updates comments and jobs are added and updated live.
  • API clean-up

    Refactors an existing API or subsystem for consistency, performance, modularization, flexibility, third-party integration, etc. May imply an API change. Frequently used during the Code Slush phase of the release cycle.

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • 🇳🇿New Zealand danielveza Brisbane, AU

    Is this still something we would want to do or has the ship sailed? Pretty big BC break.

  • Status changed to Closed: won't fix over 1 year ago
  • 🇬🇧United Kingdom catch

    Yeah I don't think this is worth it - we'd have to add bc for the service and the classes.

Production build 0.71.5 2024