Een hart voor de Drupal Community

Bij make it fly geloven we sterk in Drupal en Open Source software in het algemeen. We maken dagelijks gebruik van vele open source componenten en de vele Drupal modules die beschikbaar zijn. We dragen actief ons steentje bij aan de community door zelf patches, modules en documentatie te voorzien, daarnaast zijn enkele teamleden ook actief in de Drupal User Group vzw.

Een overzicht van onze bijdragen aan de Drupal community vind je terug op onze Drupal.org pagina. Naast het actief meerwerken aan de Drupal community, zetten we ook andere ontwikkelaars op weg door vragen te beantwoorden op Drupal Answers.

  1. Add text extractor plugins for image, file, text_with_summary and link field types

    Sven Decabooter

    @apmsooner:

    Yeah I reckoned the module would need to be updated. Was just checking if there might be ways for backwards compatibility that I didn't think about. If this refactor goes into 1.2.x branch, I guess modules that extend this functionality, would need to create a separate branch for AI 1.2.x compatibility. There is also https://www.drupal.org/project/ai_translate_paragraph_asymetric that provides a similar plugin.

    The custom FieldTextExtractor plugins for the assymetric modules could probably also be included into the ai_translate module, rather than a separate module, but then the FieldTextExtractor plugin might need some extra logic to decide whether the plugin needs to kick in or not. E.g.: - for asymmetric LB translation --> check if layout_builder_at module is enabled - for asymmetric Paragraphs translation --> check if paragraphs_asymmetric_translation_widgets module is enabled Or could that logic go into shouldExtract()? Guess not, since that's more about logic on field level, than general condition checking... Now this logic is enforced by those separate ai_translate_* modules having the dependencies set in their info.yml file. If they got included in ai_translate itself, they might be activated where they shouldn't (i.e. if translation logic is symmetric instead)

  2. Add a cleanup function

    Sven Decabooter

    @eelkeblok: I have merged issue #3519842: Support OOP hooks, so this functionality could be implemented with OOP hooks in mind. Haven't reviewed the code yet, but described functionality looks promising :)

  3. Support OOP hooks

    Sven Decabooter

    Remaining issues have been resolved, and this has been added to 1.0.x branch. @eelkeblok would be good if you could rebase your MR branch on 1.0.x and also add support for OOP hooks. Sorry for the extra work.

  4. Add Role Delegation to Drupal CMS

    Sven Decabooter

    FYI: At this point in time I would suggest to include the https://www.drupal.org/project/role_delegation into Drupal CMS, rather than https://www.drupal.org/project/roleassign, given that the former is more actively maintained, and has a wider install base.

    This only if there is a need to include anything at all, as per phenaproxima's comment above.

  5. Add text extractor plugins for image, file, text_with_summary and link field types

    Sven Decabooter

    Tried testing the MR, but the changes to the modules/ai_translate/src/FieldTextExtractorInterface.php break contrib FieldTextExtractor plugin implementations, such as in "ai_translate_lb_asymmetric" or "custom_field" module that I'm using. There is no way we can make this work without breaking backwards compatibility?

  6. Admin toolbar height is not 100% since upgrading to Drupal 11.2

    Fons

    I think it depends on how you see the sidebar vs the topbar.

    Since the sidebar holds the logo I personally feel it should always be 100% height and the header shouldn't overlap it or push it down because the logo wouldn't be in the lefthand top corner in this case which feels visually weird.

  7. Implement the current environment as a service and utilize plugins

    Sven Decabooter

    I think a plugin system for detecting the current environment would be really helpful. While a lot of people would opt to use URL matching to discover the current environment, I can see other methods being used: - a config override in settings.php - a setting in \Drupal::state() - a system environment variable (retrieved via getenv()) - e.g. set by a CI script targetting a specific environment - ...

  8. Problems when uninstalling the module

    Randal

    Hi @a8w4,

    I'm afraid I'm still not able to reproduce this ☹️ I followed these steps: - `composer create-project drupal/recommended-project:10.4.6 sandbox-10` - `ddev config && ddev start` - `ddev composer require drush/drush drupal/msqrole && ddev drush pm:enable msqrole` - `ddev drush pmu msqrole`

    I also tried installing and uninstalling via the UI, it seems to always have worked.

    Could it be possible there was just some corrupt caching or config in your project at the time? If there's any more clues as to how this could be recreated, I'm happy to look further!

  9. Support Drupal core DefaultContent API

    Sven Decabooter

    Together with the patch to default_content mentioned above, this MR allows custom_field data to be exported by default_content, and imported by default_content import OR core's DefaultContent API, used by recipes.

  10. Support Drupal core DefaultContent API

    Sven Decabooter

    A patch has been added to default_content module, to make sure the Default Content exported YML files use the UUID identifiers of referenced entities within a custom_field field, rather than their (numeric) entity ID. The patch can be found at #3532596: Add support for custom_field module

  11. Add support for custom_field module

    Sven Decabooter

    On the import side of things, there is a patch for custom_field that makes this work: #3528586: Support Drupal core DefaultContent API.

  12. Add support for custom_field module

    Sven Decabooter

    Created an MR that updates custom_field data in the Default Content export for referenced entities.

    Original output without this MR:

    field_my_custom_field:
        -
          label: 'Exported label'
          image: 21 # Some random file ID
          image__width: 1024
          image__height: 1024
          image__alt: Sunset
          image__title: ''
          file: 22 # Some random file ID
          term: 4 # Some random taxonomy term ID
    

    After applying this MR. this becomes:

      field_my_custom_field:
        -
          label: 'Exported label'
          image: aa6340f0-0006-4f31-9f2d-5595f5cce028
          image__width: 1024
          image__height: 1024
          image__alt: Sunset
          image__title: ''
          file: c36ea199-cd17-4fe5-bec7-beef1ffb51b4
          term: 78ae1ee7-f2d4-4fc0-871e-1af92730df3b
    

    The referenced UUIDs also get added as a dependency to the "_meta" definition.

  13. Add support for custom_field module

    Sven Decabooter

    Problem/Motivation

    Add support for the custom_field module, so entities using that field can be exported and imported using Default Content (or Drupal core recipe importer).

    Export functionality for most custom_field data types are working out of the box, but the ones referencing entities will export the entity target IDs, rather than UUIDs used by the importer.

    Steps to reproduce

    - Install Default Content & custom_field - Set up a custom_field field, containing (amongst others) an image field or entity reference field. - Try export / import functionality.

    Proposed resolution

    - Add support for custom_field entity references in the Normalizer class.

  14. sourceEditing plugin no longer gets enabled automatically in 1.1.x - followup

    Sven Decabooter

    Problem/Motivation

    This is a followup to #3527899: sourceEditing plugin no longer gets enabled automatically in 1.1.x. Not sure if I missed this previously, or the code has changed in the meantime, but the sourceEditing CKEditor plugin gets enabled for the "Completion" AI CKEditor plugin, but is still throwing errors for other plugins, e.g. Spellfix.

    Steps to reproduce

    - Same as in #3527899: sourceEditing plugin no longer gets enabled automatically in 1.1.x, but use Spellfix instead of Completion

    Proposed resolution

    - Fix this by enabling the sourceEditing CKEditor for the "response_text" form element in the AiCKEditorPluginBase class.

  15. Problems when uninstalling the module

    Randal

    Hi @a8w4,

    Thanks for your report! I'm unable to reproduce this issue though, would it be possible to share the exact steps / paths where this issue occurs? As well as let me know if it's reproduceable on a clean D11?

    Thanks in advance!

Sven Decabooter - Drupal Developer

"Onze teamleden bouwen zelf ook mee aan ons geliefde Drupal, en daar zijn we trots op"

Sven Decabooter
Drupal developer

Betrouwbare technologie, naadloze prestaties. Dat zijn onze Drupal-oplossingen.