Contrib.social

Need for a hook_image_path_flush

Open on Drupal.org →
Created on 26 August 2015, over 9 years ago
Updated 1 February 2023, almost 2 years ago

Problem/Motivation

Modules like adaptive_image have no way of reacting to image_path_flush();

Proposed resolution

I propose adding an extra hook at the end of image_path_flush(), like this:

 module_invoke_all('image_path_flush', $path);

Remaining tasks

  • Review and approve

User interface changes

  • None

API changes

  • New hook_image_path_flush($path) is added.

Data model changes

  • None
✨ Feature request
Status

Needs work

Version

10.1 ✨

Component
Image system  →

Last updated 1 day ago

Created by

🇲🇽Mexico rmontero CMDX

Live updates comments and jobs are added and updated live.
  • API addition

    Enhances an existing API or introduces a new subsystem. Depending on the size and impact, possibly backportable to earlier major versions.

  • Needs tests

    The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.

  • Needs subsystem maintainer review

    It is used to alert the maintainer(s) of a particular core subsystem that an issue significantly impacts their subsystem, and their signoff is needed (see the governance policy draft for more information). Also, if you use this tag, make sure the issue component is set to the correct subsystem. If an issue significantly impacts more than one subsystem, use needs framework manager review instead.

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.

  • Comment almost 2 years ago →
    🇺🇸United States smustgrave

    This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request → as a guide.

    For this new feature it will need test cases to go along with it.

    Tagged for subsystem maintainer review for their thoughts. They may have some tweaks or suggestions.

    Did not review code.

  • Status changed to Closed: duplicate about 1 month ago
  • Comment about 1 month ago →
    🇩🇰Denmark arnested

    I'm pretty sure this is fixed / handled by the change in #2833129 → .

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024