Account created on 23 January 2020, almost 5 years ago
#

Recent comments

🇦🇷Argentina abelpzl

It seems that the work to make this module compatible with Drupal 11 was concentrated on Automated Drupal 11 compatibility fixes for dashboards 📌 Automated Drupal 11 compatibility fixes for dashboards Needs review
Should we close it?

🇦🇷Argentina abelpzl

Are more reviews required in order to approve the MR?
Great work!

🇦🇷Argentina abelpzl

The MR3 makes the module compatible with Drupal 11 and works as expected. I also tested it on Drupal 10.4.

RTBC +1

🇦🇷Argentina abelpzl

Hi everyone!
Are the issues detailed here still blocking a stable release?

🇦🇷Argentina abelpzl

I think a notice should be put on the module page to warn users that this module is not compatible with php 7.4.
This would help avoid problems on sites that cannot yet upgrade to php 8.x / Durpal 10

The patch solves the problem in my case.

Cheers!

🇦🇷Argentina abelpzl

Hello!

Thanks for your interest in this module.

I completely forgot to create a development release. The development release is now available on the module page.
I hope to be able to release an alpha version soon, but first I want to solve some problems in the pipeline.

I'm going to work on this module this week since I have some free time.

Thank you

-

https://www.drupal.org/project/zip_field_files/releases/1.0.x-dev

🇦🇷Argentina abelpzl

Clarify that the example needs the  FormAlter as Plugin  module to work

🇦🇷Argentina abelpzl

There are still issues blocking the release of a stable version.
I don't think a stable release is possible without first finishing the tasks detailed in " Stable release roadmap ".

🇦🇷Argentina abelpzl

I ran into the same problem on drupal 10.3.1 and paragraphs 1.17.
The merge request made by Codebymikey solved my problem.

🇦🇷Argentina abelpzl

I have the same problem testing 3.0.0-alpha3.
Proposed solution work. Now the "Simple hierarchical select" format is selectable again in the field presentation.
I think MR should be implemented as soon as possible.
-
Is it a good idea to add functional test cases for FieldFormatter and FieldWidget plugins?
-
Thanks.

🇦🇷Argentina abelpzl

Hello everyone! I think that the question raised by @tunic is correct, I think that the great work done by @Increweb21 should be found on another page with documentation related to the Paragraphs module. Mostly to keep this documentation as simple as possible.

🇦🇷Argentina abelpzl

Fix error in "States based on boolean form fields (checkboxes)" code example.

🇦🇷Argentina abelpzl

Investigating the problem I realized that the following error was being displayed in the DevTools console

drupal.js?v=10.2.4:64 Uncaught TypeError: Cannot read properties of undefined (reading 'include_path')

On line 525 of the file leaflet/js/leaflet.drupal.js :

lFeature = this.create_polygon(feature, map_settings ? map_settings['leaflet_markercluster']['include_path'] : false);

An error occurs if $map_settings['leaflet_markercluster'] does not have a value.
-
Fix the problem by adding 'leaflet_markercluster' => ['include_path' => FALSE, 'control' => FALSE], in 'mymap' configuration.

function my_modulo_leaflet_map_info() {
  return [
    'MyMap' => [
      'label' => 'My Map',
      'description' => t('My map'),
      'settings' => [
        'center' => [
          'lat' => '-40.41',
          'lon' => '-64.69',
        ],
        'dragging' => TRUE,
        'touchZoom' => TRUE,
        'scrollWheelZoom' => TRUE,
        'doubleClickZoom' => TRUE,
        'zoomControl' => TRUE,
        'attributionControl' => TRUE,
        'trackResize' => TRUE,
        'fadeAnimation' => TRUE,
        'zoomAnimation' => TRUE,
        'closePopupOnClick' => TRUE,
        'layerControl' => TRUE,
        'layerControlOptions' => [
          'position' => 'topright',
        ],
        'leaflet_markercluster' => ['include_path' => FALSE, 'control' => FALSE],
      ],
      'layers' => [
        'earth' => [
             // ...
          ],
        ],
      ],
      'plugins' => [],
    ],
  ];
}

-
I think this should not be a required parameter.

🇦🇷Argentina abelpzl

hI!
There is still a lot of work to be done to get this theme working properly, it is in a very early stage of development.
I'm going to put a notice on the project page to let people know that it's not usable yet.
In the meantime I'm going to work on it.
-
Thanks for your interest :)

🇦🇷Argentina abelpzl

I am facing a similar problem
I need the folder where a file is saved to be formed with the id of a node related to the main node.
Something like this:
[node:field_a:entity:field_b:entity:field_c:entity:nid]
This works correctly in the drupal 7 version of this module but not in 8.x

🇦🇷Argentina abelpzl

Is there anything new?
What are the problems that need to be resolved for a stable version?

🇦🇷Argentina abelpzl

I am working on a migration of a site created in Drupal 7 to Drupal 10.
I use this module to create the path to a file with a custom token of type node. This token is responsible for determining in which path the file should be saved depending on the information on the node.
When importing the migrations I get an error as I do not have the files available in this instance.
It is not the same error as described in this issue but I believe it is related.
I think this module should ignore the entities that are being synchronized. It may be best to check it in filefield_paths_entity_insert().
I am attaching a patch

🇦🇷Argentina abelpzl

My problem was fixed by adding zoomControlPosition => topleft in the definition of my HOOK_leaflet_map_info().

function my_module_leaflet_map_info() {
  return [
      'my_map' => [
        'label' => 'My Argentine map',
        'description' => t('...'),
        'settings' => [
          // ...
          'zoomControlPosition' => 'topleft',
          // ...
        ],
      ],
      'layers' => [ ... ],
      'plugins' => [ ... ],
    ],
  ];
}

The following code works correctly in my case and marks three points on my map

$map_features = [
  0 => [
    'type' => 'point', 
    'lon' => '-60.292969', 
    'lat' => '-36.315125'
  ],
  1 => [
    'type' => 'point', 
    'lon' => '-63.632813', 
    'lat' => '-32.287133'
  ],
  2 => [
    'type' => 'point', 
    'lon' => '-65.566406', 
    'lat' => '-37.195331'
  ],
];

$map_info = \Drupal::service('leaflet.service')->leafletMapGetInfo()['my_map'];
$build['map'] = \Drupal::service('leaflet.service')->leafletRenderMap($map_info, $map_features, '450px');
🇦🇷Argentina abelpzl

I share a patch applicable to Drupal 10.2.2
I based it on the previous patches. Add a "Clear filter" button.
This patch solved my problem, but I haven't tested it enough to ensure it works correctly in all cases.

Is it possible that this filtering functionality will be part of the Core in the future?

🇦🇷Argentina abelpzl

I almost drove myself crazy trying to figure out why the empty field tag kept showing up. 😂
Thanks to #8 I understood what the problem was, by disabling branch debug mode the tag was finally hidden.
Shouldn't this work differently?

🇦🇷Argentina abelpzl

I had the same problem without using the Layout Builder module and I solved it by applying #2.
Can someone merge this? how can I help?

🇦🇷Argentina abelpzl

Patch #46 could not be applied in admin_toolbar 3.4.1 and drupal 9.5.10.
Instead #48 was applied correctly and seems to work correctly.

🇦🇷Argentina abelpzl

I share my solution to the errors named in #28 Taxonomy overview list with exposed filter Active
This patch is applicable to D9.5.3
Keep in mind that in this patch I change the word comparison criteria with respect to the other patches.
-
In the future I would like to work on a custom module that adds the search to the list of terms. For the moment this solves my problem with a very long list of terms.

🇦🇷Argentina abelpzl

Testing 2975863-26-taxonomy-overview-filter.patch found it works well overall but reloading the form after performing a search produced the following errors:

Notice: Undefined offset: -1 in Drupal\taxonomy\Form\OverviewTerms->buildForm() (line 200 of core/modules/taxonomy/src/Form/OverviewTerms.php).
Notice: Undefined index: term in Drupal\taxonomy\Form\OverviewTerms->buildForm() (line 258 of core/modules/taxonomy/src/Form/OverviewTerms.php).
Notice: Trying to access array offset on value of type null in Drupal\taxonomy\Form\OverviewTerms->buildForm() (line 258 of core/modules/taxonomy/src/Form/OverviewTerms.php).

🇦🇷Argentina abelpzl

#42 is applied correctly in 9.5.3 and in my opinion works as expected.

Thanks for providing this solution.

🇦🇷Argentina abelpzl

#142 does work for me in 9.5.3 with php 7.4

Production build 0.71.5 2024