Een hart voor de Drupal Community

New tag has been released. Thanks again!

Hi @andybroomfield,

Good catch! The class name was altered at the last moment before merge, seems I forgot to alter the service name. Thank you.

I'll merge and release a new version.

randalv made their first commit to this issue’s fork.

Seems logical, I've added the hooks_converted parameter.

Since the other threads are resolved, I'll merge into 1.2.x

Sven Decabooter added a comment on drupal.org: Support OOP hooks

Sven Decabooter added a comment on drupal.org: Support OOP hooks

MR looks good to me. Maybe we also need to add the 'hooks_converted' parameter for performance improvement: https://www.drupal.org/node/3490771

Seems like core is mostly putting all hooks together in a Hook\[MODULENAME]Hooks.php file, but I agree splitting up in logical chunks / files makes sense.

Processed the feedback.

I was also debating whether the Help and Cron hooks could be merged into a "Core.php" file since they're both small hooks, but I just kept them in their own file. Figured it's better to keep things as split up as possible, which is the whole point of moving away from one gigantic .module file to OOP hooks :)

- Hook help still needs to be added as OOP hook - All hooks need to be marked as converted, as per https://www.drupal.org/node/3490771

Sven Decabooter added a comment on drupal.org: Improve performance by preventing unnecessary scanning of procedural hooks

Problem/Motivation

See https://www.drupal.org/node/3490771

svendecabooter created an issue. See original summary.

Reverted, because module still contains a preprocess procedural hook.

Problem/Motivation

See https://www.drupal.org/node/3490771

Problem/Motivation

See https://www.drupal.org/node/3490771

Problem/Motivation

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Problem/Motivation

Starting in Drupal 10.2.0, plugins should be defined with PHP attributes instead of annotations. When defining new plugin instances you should prefer PHP Attributes over annotations assuming the underlying plugin manager has been updated to support the use of attributes.

Proposed resolution

Allow PHP attributes for plugin definitions, as per https://www.drupal.org/docs/drupal-apis/plugin-api/attribute-based-plugins

Randal added a comment on drupal.org: Support oop hooks

Problem/Motivation

Implement OOP hooks for better OOP & DI support, instead of having all hook logic in the module file.

Proposed resolution

Convert hook implementations to OOP hooks, with backwards compatibility. See https://www.drupal.org/node/3442349

Remaining tasks

- Define hooks that can be converted into OOP hooks - Create hook clas(ses) to logically bundle relevant hooks with their logic - Add backwards compatibility support

User interface changes

N/A

API changes

N/A

Data model changes

N/A

randalv created an issue. See original summary.

@svendecabooter the changes have been applied in the MR. Feel free to check it out, everything seems to work fine.

Changed minimum Drupal Core version to 10.1, PHP remains at >= 8.1.

randalv made their first commit to this issue’s fork.

I created a dev release

Bram Goffings added a comment on drupal.org: Add gitlab-ci file

Bram Goffings added a comment on drupal.org: Abstract token usage

Problem/Motivation

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

aspilicious created an issue. See original summary.

This merge request, together with https://www.drupal.org/project/ai_provider_openai/issues/3519302 Allowed me to create this module: https://www.drupal.org/project/ai_usage_limits

AI usage limits is a Drupal module that extends the Drupal AI module by allowing administrators to define usage limits for each enabled AI provider.

Features

Supports the following usage limits:

• Input token usage
• Output token usage
• Total token usage
• Cached token usage
• Reasoning token usage

Post-Installation

Admin UI available at `/admin/config/ai/usage_limits`.

Additional Requirements

• Depends on the Drupal AI module.
• Needs several patches to work at this moment
  • https://www.drupal.org/project/ai/issues/3517618
  • https://www.drupal.org/project/ai_provider_openai/issues/3519302