Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Apparently beta means something different in the Drupal console camp because it certainly does not mean that people should be integrating with it b/c they have drastically changed the API and we need to rework the command.
* Currently it no longer registers as a command.
* There also appears to be several classes that are no longer available that we are using.
Last known version of drupal console that the command worked with beta2
Comments
Comment #2
grndlvl CreditAttribution: grndlvl at GollyGood Software for Zivtech commentedComment #3
grndlvl CreditAttribution: grndlvl at GollyGood Software for Zivtech commentedFirst pass.
@todo:
Comment #4
grndlvl CreditAttribution: grndlvl at GollyGood Software for Zivtech commentedThis is because errors are being suppressed somewhere and
Filesystem::mkdir()
is usingerror_get_last()
to check if it should re-throw one of their own suppressed errors... this will be fun...Comment #5
jlamp CreditAttribution: jlamp at GollyGood Software commentedWorking from #4, the only errors I got from trying to run the Install command were from the using statements being outdated. Just added /Core/ in the Drupal Console using statements for Command and Styles, and #4 (with uncommenting the code that actually downloads the files) worked great.
Patch Submitted.
Comment #6
jlamp CreditAttribution: jlamp at GollyGood Software commentedComment #7
grndlvl CreditAttribution: grndlvl at GollyGood Software for Zivtech commentedThis is looking great. The only part that doesn't seem to be working now is that the messages are not displaying correctly only their ids are showing.
i.e.
Should say "'Downloading CKEditor package version %s...'" where "%s is the version name
These use to translate via Drupal console by placing the text in console/translations/en/ckeditor_media_embed.install.yml and calling
$this->trans('commands.ckeditor_media_embed.install.messages.comment-downloading-package');
in code.Comment #8
grndlvl CreditAttribution: grndlvl at GollyGood Software for Zivtech commentedComment #9
jlamp CreditAttribution: jlamp at GollyGood Software commentedPatch Submitted.
The class needed an annotation for it to work. Added the Annotation using statement (and also sorted using statements alphabetically so it's easier to read) and added in the annotation, now the strings are being translated correctly.
Comment #11
grndlvl CreditAttribution: grndlvl at GollyGood Software for Zivtech commentedAwesome sauce, committed! Thanks!