Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
image system
Priority:
Normal
Category:
Task
Assigned:
Issue tags:
Reporter:
Created:
9 Oct 2013 at 10:37 UTC
Updated:
29 Jul 2014 at 23:02 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
claudiu.cristeaPatch.
Comment #2
dawehnerWe should at least adapt the existing implementations as well. They work because we now have magic loading of annotations.
Comment #3
dawehnerIf you apply the patch there is still some
use Drupal\system\Annotation\ImageToolkit;statements in core.Comment #4
claudiu.cristeaHere with cleaning of use statements.
Comment #5
claudiu.cristeaDid it again :). Interdiff to #1.
Comment #6
claudiu.cristeaRight patch
Comment #7
fietserwinI did a global search on " extends plugin" to find all annotation classes. They are always in an Annotation subdirectory. Most of the times they are under an Annotation subdirectory of the directooy where the plugin type itself can be found. The only exception is Action which is directly under /Core/Annotation instead of /Core/Action/Annotation.
So the file should be moved to Core\ImageToolkit\Annotation. and the namespace should be adapted accordingly.
Comment #8
claudiu.cristeaOK. Moved under Annotation/.
Also changed priority to 'normal' as this blocks #2073759: Convert toolkit operations to plugins.
Comment #9
mondrakeUnder Annotation/ makes it more understandable, I believe.
Grepped the codebase after applying #8:
uses of the annotation class in the toolkit plugins (3 of them, GDToolkit, TestToolkit and BrokenToolkit)RTBC
Comment #10
webchickCommitted and pushed to 8.x. Thanks!