🇩🇪Germany @vistree

Hi @jnicola,
this is what I did:

1. Create new empty Mysql database
2. Install Drupal 10:
   composer create-project drupal/recommended-project:10.3.8 simplegroup
3. Add repository to composer.json
   

   {
               "type": "vcs",
               "url": "https://git.drupalcode.org/project/simple_grouped_content.git"
           }

4. Allow dev/alpha modules:
   

   composer config minimum-stability dev
   composer config prefer-stable true

5. Add simple_grouped_content:
   

   composer require drupal/simple_grouped_content
   

6. Run installer in webbrowser:
   http://simplegroup.local/core/install.php
7. Add database settings to installer

Install fails with

An AJAX HTTP error occurred.
HTTP Result Code: 200
Debugging information follows.
Path: /core/install.php?rewrite=ok&profile=simple_grouped_content&langcode=en&id=1&op=do_nojs&op=do
StatusText: OK
ResponseText: Drupal\Core\Entity\EntityMalformedException: All group roles require a scope. in Drupal\group\Entity\GroupRole->preSave() (line 341 of /var/www/html/simplegroup/web/modules/contrib/group/src/Entity/GroupRole.php).

I tried with Drupal 10.0, 10.2 and 10.3

@aleix,
should your hook work with group 3.3.x version? For me it seems that the group ID is not available within media library ...
I am using:
- group: 3.3.0
- group media: 4.0.3
- media_library: 10.3.8

Hi, I tried the above installation with Drupal 10.3 and simple_grouped_content 2.0-alpha1
Should I try the 1.2.x version instead?

@jnicola, can you tell me which Drupal version you last used to install the environment? Drupal 9, 10.0, 10.1, ...?
And, which group version was used in your initial local install?

I needed to use

composer config minimum-stability dev
composer config prefer-stable true

to install the 2.0-alpha1

+ group 3.3 is the recommended version on new installs - should simple_groupd_content work also with th 3.x branch of group?

Hi, trying to use command from project page
composer require drupal/simple_grouped_content
results in

  Could not find a matching version of package drupal/simple_grouped_content. Check the package spelling, your version constraint and that the package is available in a stability which matches your minimum-stability (stable)  

Hi @dww,
can you explain why you use "User ID from route context" as contextual filter - instead of using "Group Ids from current user".
I am still searching for a better group management. If I don't find existing stuff (as a distribution) I will setup all needed views on my own.
And I want to use the best performing approach ;-)

vistree → created an issue.

vistree → created an issue.

Hi, I try to get gulp running on a Mac M3. I tried the old approach using patch from #2. But I can't get it to work.
Can anyone give a quick step by step on how to make gulp work on a Mac with M3 processor?
NPM version?
NODE version?
Python version?
...

Thank you so much!

vistree → created an issue.

Same problem here. Seems, like the fix from MR solves the problem. Will this be merged soon?

Thanx @drubb,
that helps and solves the problem for me ;-)

vistree → created an issue.

Thanx @damienmckenna

This is a really old issue, but can we add #access = false on field_meta_tags which is a container? Or will this remove all field-data within that group if a user edits node with the access=false set?

I found a possible solution: instead of using hook_form_alter() I can add a description using hookp_reprocess_details(&$variables)

/**
 * Implements hook_preprocess_HOOK().
 */
function mymodule_preprocess_details(&$variables)
{
  if (isset($variables["element"]["#id"]) && $variables["element"]["#id"] == 'edit-scheduler-settings') {
    $variables["description"] = 'Info when cronjobs are about to run';
  }
}

vistree → created an issue.

Hi Daniel,
if it helps: I found a second problem with Gin, Content Moderation and editing existing nodes. I where able to fix using https://www.drupal.org/files/issues/2021-12-27/2991986-16.patch → from
https://www.drupal.org/project/drupal/issues/3358471 🐛 TypeError: Cannot access offset of type string on string Active
Besides this current Gin version now works fine for me

I see the same problem if I combine content moderation and Gin theme. After latest update and using PHP 8.1 I can't save any existing content anymore. Using patch from #9 solves the issue for me.
Using Claro instead of Gin works without any problems
Using Gin without content moderation also works

So, it seems to be the combination.

I don't know if this is the correct fix - as there are no problems with other themes like Clar0. But https://www.drupal.org/files/issues/2021-12-27/2991986-16.patch → from https://www.drupal.org/project/drupal/issues/3358471#comment-15038304 🐛 TypeError: Cannot access offset of type string on string Active (patching core WidgetBase.php) seems to solve my problem.

vistree → created an issue.

Hi @daniel.bosen,
this does perfectly work!! I did absolutly didn't know about the composer "as" option. Very cool. Thanx!
Should I close this issue or do you want it to stay for upcoming update of Gin within thunder-distribution?

vistree → created an issue.

Same problem here - you can only subscribe to "forum" if you go to the related taxonomy page ...

Sorry, I don't see the error. Maybe it is related to parent::fetchNextRow(); ??

@anybody - for my understanding: currently the non-vue version is used in Drupal cookies module, right?
I am not javascript / vue developer. So I don't know if I am the right person to create a fork. Sure, I can remove all the _blank values. I will try to contact jfeltkamp

We have the same problem with the library - _blank should not be used on accessible websites.
As both projects (vue and non-vue) are on github - can't we just fork them? For the non-vue version, there is a License.txt and it seems to be OK to fork as long we keep correct attribution.

/** @license CookieJSR v1.0.10
 * cookiejsr.min.js
 * cookiejsr.min.css
 *
 * Copyright (c) Joachim Feltkamp, Hamburg, Germany.
 *
 * This source code is licensed under the CC BY-ND license found in the
 * LICENSE file in the root directory of this source tree.
 */

Source code is licensed under the CC BY-ND license.
Creative Commons Attribution-NoDerivs 3.0 Germany License.
https://creativecommons.org/licenses/by-nd/3.0/de/

You are free to:

    Share — copy and redistribute the material in any medium or format
    for any purpose, even commercially.

    The licensor cannot revoke these freedoms as long as you follow the license terms.

Under the following terms:

    Attribution — You must give appropriate credit, provide a link to the license, and indicate if changes were made.
    You may do so in any reasonable manner, but not in any way that suggests the licensor endorses you or your use.

    NoDerivatives — If you remix, transform, or build upon the material, you may not distribute the modified material.

    No additional restrictions — You may not apply legal terms or technological measures that legally restrict others
    from doing anything the license permits.

Notices:

    You do not have to comply with the license for elements of the material in the public domain or where your use is
    permitted by an applicable exception or limitation.
    No warranties are given. The license may not give you all of the permissions necessary for your intended use. For
    example, other rights such as publicity, privacy, or moral rights may limit how you use the material.

On Drupal > 10.2 the patch from #13 brings an error when running drush updb:
Drupal\content_translation\Plugin\views\filter\UnTranslated::operators() must be public

Attached find a minimal change for the patch from #13

Hi @aaron.ferris,
Thank you for your feedback. It makes me aware on a patch I use on Drupal core from https://www.drupal.org/files/issues/2020-07-15/drupal-add_views_untransl... →

So, this ticket can be closed, as the problem is not related to Drupal core in my case.

Hm, I found an error regarding Seven theme (I still use the community module). After installing the patch from https://www.drupal.org/project/seven/issues/3267440#comment-15027460 📌 Add ckeditor5-stylesheets config RTBC I were able to save my text editor config without problems

Same problem here. Can't save my Basic HTML text editor configuration after updating to 10.3.2

This value must be a number of bytes, optionally with a unit such as "MB" or "megabytes". does not represent a number of bytes.

vistree → created an issue.

Will this fix be result in a new beta1?

New patch rerolled against 2.0.4

#3 fixed the problem also for me!!

Hi, may I come back to this issue? If I add a charts field to one of my content types, I can already select "Highmap" as charting library. But then it does not work anymore and I find some errors in the browser console:

jquery.min.js?v=3.7.1:2 
 POST http://mysite.local/de/node/6328/edit?destination=/de/admin/content&ajax_form=1&_wrapper_format=drupal_ajax&_wrapper_format=drupal_ajax 500 (500 Service unavailable (with message))
send	@	jquery.min.js?v=3.7.1:2
ajax	@	jquery.min.js?v=3.7.1:2
q.fn.ajaxSubmit	@	jquery.form.js:341
Drupal.Ajax.eventResponse	@	ajax.js?v=10.2.4:796
(anonymous)	@	ajax.js?v=10.2.4:646
dispatch	@	jquery.min.js?v=3.7.1:2
v.handle	@	jquery.min.js?v=3.7.1:2
handleMouseUp_	@	unknown

ajax.js?v=10.2.4:1219 Uncaught 
Drupal.AjaxError {message: '\nEin AJAX-HTTP-Fehler ist aufgetreten.\nHTTP-Rückga…encountered an unexpected error. Try again later.', name: 'AjaxError', stack: 'Error\n    at http://mysite.local/core/misc/ajax.js?v=…ttp://mysite.local/core/misc/ajax.js?v=10.2.4:1916:3'}
message
: 
"\nEin AJAX-HTTP-Fehler ist aufgetreten.\nHTTP-Rückgabe-Code: 500\nIm Folgenden finden Sie Debugging-Informationen.\nPfad: /de/node/6328/edit?destination=/de/admin/content&ajax_form=1&_wrapper_format=drupal_ajax\nStatustext: 500 Service unavailable (with message)\nAntworttext: The website encountered an unexpected error. Try again later."
name
: 
"AjaxError"
stack
: 
"Error\n    at http://mysite.local/core/misc/ajax.js?v=10.2.4:196:32\n    at http://mysite.local/core/misc/ajax.js?v=10.2.4:1916:3"
[[Prototype]]
: 
Error at http://mysite.local/core/misc/ajax.js?v=10.2.4:196:32 at http://mysite.local/core/misc/ajax.js?v=10.2.4:1916:3
Drupal.Ajax.error	@	ajax.js?v=10.2.4:1219
complete	@	ajax.js?v=10.2.4:608
M.complete.M.complete	@	jquery.form.js:302
c	@	jquery.min.js?v=3.7.1:2
fireWith	@	jquery.min.js?v=3.7.1:2
l	@	jquery.min.js?v=3.7.1:2
(anonymous)	@	jquery.min.js?v=3.7.1:2
load (async)		
send	@	jquery.min.js?v=3.7.1:2
ajax	@	jquery.min.js?v=3.7.1:2
q.fn.ajaxSubmit	@	jquery.form.js:341
Drupal.Ajax.eventResponse	@	ajax.js?v=10.2.4:796
(anonymous)	@	ajax.js?v=10.2.4:646
dispatch	@	jquery.min.js?v=3.7.1:2
v.handle	@	jquery.min.js?v=3.7.1:2
handleMouseUp_	@	unknown

Hi @andileco,
Thanx for producing the video tutorial. This is perfectly helpful and I now understand how to use this module.
Just 3 additional questions:
1. can I use both, highmaps and highchart library in one Drupal page? Do I really need ti change the library on charts config page to highmaps to make the views type work?
2. As JSON source, can I use everything from https://code.highcharts.com/mapdata/ ?
3 is it possible to use Tiled Web Maps (https://www.highcharts.com/docs/maps/tiledwebmap)?

Thank you so much for your great support !

Hi, I really tried hard to get this module run. But, I am lost ... Any documentation about how to set this up (taxonomy, how to add map source, how to add geolocation to content types, ...) would be really helpful!

After some debugging I found out that the error is not related to the single_content_sync module - instead it had to do with the charts structure. I fixed that and now I can export the chart_config by just adding the field name to

/modules/contrib/single_content_sync/src/Plugin/Derivative/SingleContentSyncFieldProcessor/SimpleFieldDeriver.php

vistree → created an issue.

Hi, the solution from @jim.shreds does not work for me. I tried:

/**
 * Implements hook_preprocess_page().
 */
function ditrare_preprocess_page(&$variables) {
  $variables['#attached']['drupalSettings']['responsive_menu']['custom'] = [
    'options' => [
      'navbar' => [
        'add' => TRUE,
        'title' => 'This is a test',
      ],
      "offCanvas" => [
        "position"=> "left",
      ]
    ],
    'config' => [
      'classNames' => [
        'selected' => 'my-custom-menu--active-trail',
      ],
    ]
  ];
}

What I really want is to use the mmenu option left-front or right-front - so that the offcanvas is overlaying the content instead of moving the content from the side to make space for the offcanvas menu.

Should the "offCanvas" option work within theme-page-hook? + Should the position: right-front work?

Thanx @tank,
Then it is as expectected. I thought that there could be an option within gui or alter hook

vistree → created an issue.

Thank you Grevil ;-)

Using about:blank results in console errors on Chrome. Using a valid placeholder (link in patch from #8) will make sure that there is no HTML error and no Chrome console error.
I think this issue should be reopened and the solution from #8 should be proved.

Thanx @Grevil and @Anybody,
my MR now was build successfully.

vistree → created an issue.

@Anybody: can you have look at the MR? Errors seem not to be related to my changes …

To be honest ... I even did not realize, that the smaller text is there and can do what I was missing ;-)
So, yes. You are right ;-) We should make the "Video only" accept link a button and make the "Accept all cookies" less prominent - or we can also remove it??
What do you think?
I can then try to create a MR ;-)

@Anybody - sure: on the attached screenshot you find the standard video filter overlay (cookies-filter-overlay-button.png). The button "Accept all Cookies" will trigger all cookis from all "groups".
So, it is the same behaviour as in the main cookies dialog (cookies-main-dialog.png)
From my point of view it should instead only set the cookie for the "video" group (cookie-video-dialog.png)

Can one of the maintainers please explain, if the "Alle zulassen" / "Accept All" is really what we want to display as button overlay to filtered content?
What do others think? Should we "accept all" cookies with this button or should we only accept the video related cookie settings?

I updated the patch from #161 to work with Drupal 10.2.3. Only one small change within core/modules/jsonapi/tests/src/Functional/NodeTest.php

By the way: I am also interested in a Drupal 7 compatible patch. Is someone already working on this?

The patch also works fine on my sites. I installed it on one page with > 1000 spams per day - and the spams have gone while wanted webform submissions still arrive.

Thanx @drubb, that helps and solves my needs ;-)

Hi, can patch from #10 been merged + a new D10 compatible version been created?

Can someone explain why the D10 compatibility patch is not merged? Are there outstanding problems?

vistree → created an issue.

vistree → created an issue.

No activity here. I think I will remove the module and look for a different one.

Thanx @Wim Leers !!

This is still a problem - also in Drupal 10. Any update on how to solve this issue?

The patch from #11 still applies and fixes the problem also in Drupal 10

We have the same problem on Drupal 10 - and I will try patch from #11 and report back

hi, it seems, that there is still a small outstanding issue: if the menu is translatable and the menu_item_extra fields are translated - always the source language is returnde - and not the translation

vistree → created an issue.

Hi @nikathone,
can you share your solution? I also need to enable pages - on the main page and the attachment.

Can MR from #15 be merged - or are there outstanding problems?

vistree → created an issue.

I am still struggling with the migration of the multilingual field collections. Does anyone here have any advice for me?

The patch attached should apply to 6.1.2 now.

Sorry, I accidently pressed the "create MR" in this ticket - without even downloading the branch ;-(
What do I exactly need to do? In the branch I can't find any of the patches shown in my project. I guess, they are created by the alma script when installing the local Drupal 9 site and starting the initial migration.
Can I add the patch as normal to the composer.json within the root folder of the branch?

Hi Vlad,
thanx again ;-) My question was more about existing migration ymls after creating the initial D7 to D10 migration files.
E.g. for my D7 field-collection field_collection_education I find
1. migrate_plus.migration.upgrade_d7_field_collection_education.yml
2. migrate_plus.migration.upgrade_d7_field_collection_revisions_education.yml

From your information in #7 I now edited file from 1. -> migrate_plus.migration.upgrade_d7_field_collection_education.yml

I did all the modifications and AFTERWARDS I did a copy of that file - renamed it to
3. migrate_plus.migration.upgrade_d7_field_collection_education_trans.yml

and did the modifications described in #7

I now have 3 files!!!! And run the migration for all 3 of them.
The result seems not to be OK ;-(

Am I doing something wrong? Do I need to exclude file 2. (migrate_plus.migration.upgrade_d7_field_collection_revisions_education.yml) ?

It seems to be a problem with paragraph migration itself. Everything on my side seems to be OK.
There is a proposed solution in https://www.drupal.org/project/paragraphs/issues/3160679 ✨ Field Collection Node content translation source plugin Needs review

As the documentation of Acquia Migrate points out that it will provide multilingual installations I thought that the migration of multilingual field_collections should work out of the box ;-)

Hi Wim,
I don't really understand what you want me to do. It seems enough to make sure to use the latest patch from https://www.drupal.org/project/focal_point/issues/3267350 ✨ Migrate Focal Point module from D7 to D9. RTBC
Acquia Migrate adds patch from #28
I replaced it with patch from #31 - and everything works fine ;-)

The other requirements seem to be related to my special D7 source. No real need to debug that one.

vistree → created an issue.

vistree → created an issue.

And a second question: do we need to also touch the revisions migration files?
E.g. I have
- migrate_plus.migration.upgrade_d7_field_collection_education.yml
- migrate_plus.migration.upgrade_d7_field_collection_revisions_education.yml

+ for my migrations, the new paragraph field will have translation turned on (true). Do we need to change this or will this automatically be correctly set by the new plugins?

Hi @vlad.dancer,
thanx for your description!!! One question: in

process:
  type:
    -
      plugin: migration_lookup
      migration: upgrade_d7_field_collection_type
      no_stub: true
      source: field_name

where does the source: field_name come from? What to use? Is it the same fieldname defined in the source section? Or should it always be static "field_name"?

@abramm, sorry for confusing ... I tested the DEV-version mentioned in #6 and thought it was output of your MR ;-)

Hi abramm,
thanx for your patch - works fine for me with Drupal 10 ;-)

Hi nmangold,
thanx for your reply. I know about those classes. What I need is an idicator, if the control is on the first / last slide. But, in the meanwhile I don't work on the old project anymore.

Hi @andileco,
your MR looks really good. From my point of view this will really good default values ;-)

Hi @andileco,
I tested the DEV-version of the charts module and I can now create bubble charts with 3 data values.
Thanx a lot ;-)

Thanx @codebymikey for the D10 compatible release ;-)

Hi andileco,
thanx for your commit! What does it exactly mean regarding my question? Is it now possible to add z-values?

Any help? Or should bubble charts be used only within charts? If that is the case, we should remove the bubble option from charts node field widget!

vistree → created an issue.

@Anybody - the merge request shows "The source branch is 1 commit behind the target branch" Should I do the rebase - or is this something you should do?
If I should do - do you normally use the normal rebase option or the "rebase without pipeline" option?

@Anybody - sure - I will do ;-) Do you want me to create it against 1.2.x-dev branch?

Ok, after creating a fresh new diagramm and directly using scatter, that one works. Before I changed an existing diagramm from pie to scatter - and that seems not to have worked ...