paris
Account created on 13 May 2014, over 11 years ago
#

Merge Requests

More

Recent comments

🇫🇷France lazzyvn paris

I don't understand what you mean, because the two modules are completely different, as well as their logic. If you like the logic and configuration of the entity browser, you can use it. I am closing this request, because I am sure I cannot implement the logic from the entity browser.

🇫🇷France lazzyvn paris

The main objective of this module is to replace the entity browser module by creating an entity reference view and mapping it in the field settings.

🇫🇷France lazzyvn paris

Oh I think fixed it is not complicate
in widget/formatter looking for target_bundles just check handler_settings[negate] and use array_reverse.

🇫🇷France lazzyvn paris

Oh, it's up to you, I have no opinion on adding options. I thought about it at first, but here's the technical difficulty. There are really a lot of options for Bootstrap-tables / Datatable. When you make form configuration for module formatter, you can't create a variable with multiple options. These options depend on the type of table libraries you want to display.
But If you don't need the functionality of bootstrap tables or datatables don't active option, the default Drupal table will be displayed by default.
And after years and many sites using it, this is the first time I see asking this. I think in reality, not many people want to use this functionality.
Could you help rewrite the document in readme.md?

🇫🇷France lazzyvn paris

You can active module 'Paragraphs Type Permissions'. You can set role to see button add or use css to hide it

🇫🇷France lazzyvn paris

Beaware/!\ The paragraph table is designed for a single paragraph type. You can select multiple paragraphs, but only the first one will be selected for display.

🇫🇷France lazzyvn paris

You can select another library or without bootstrap table use your own theme.

🇫🇷France lazzyvn paris

It is designed with permission to add paragraphs

🇫🇷France lazzyvn paris

why mr? do you review this?
for me this patch is incorrect. I can accept this, you can see it create phpcs warning. and the translate must do in markup render

🇫🇷France lazzyvn paris

@colan it fixed check dev version

🇫🇷France lazzyvn paris

Committed and fixed in another ticket, your merge request is not validated for phpcs

🇫🇷France lazzyvn paris

It's up to you. if you base on module entityreference_modal_create you can save your time 10% with form widget settings, or you can create a patch to add new options to the widget's config or create new widget entity_reference_modal.
You need to rewrite the js library by yourself with the same features as bootstrap table. Honestly, up to now I still haven't seen any opensource library that is as good as bootstrap-table. I think you can try with datatable or hansontable. If it's too heavy, you can write it manually as long as it can paginate the table, filter on table cell, First column have to select 1 items (radio) or multiple checkbox foreach lines. If you can replace bootstrap table library, you don't need to load bootstrap 5 cdn.

🇫🇷France lazzyvn paris

The instructions of the module can't help you install and configure?
Requesting a demo site to try will never be available because it uses api key of gemini, deepseek, openai,... or selfhost. Most of the sites of my customers are using chatbot AI + RAG for ERP application intranet and self-host AI model so it will definitely not be public

🇫🇷France lazzyvn paris

If your theme is not base on Bootstrap 5 then there is an option to use Bootstrap CDN. If you select this option your popup will work fine.
Without Bootstrap lib ? this definitely cannot happen as you wish. It is designed with Bootstrap table library.

🇫🇷France lazzyvn paris

Tested by myself, it works fine, but I can't reproduce this bug. Try the development version, it may be fixed in another issue.

🇫🇷France lazzyvn paris

Tested by myself, it works fine, I can't reproduce this bug. Try the dev version, maybe it is fixed in another issue

🇫🇷France lazzyvn paris

this patch is not good at all
If an entity is not a node then it has to run a foreach all rows this is not optimal


  $ids = [];

  foreach ($rows as $key => $row) {
    $revision = $row->_entity;
    // Get the *node ID* from the revision entity.
    if ($revision->hasField('nid')) {
      $nid = $revision->get('nid')->value;
      $ids[$key] = $nid;
    }
  }
  // Get unique model values and their original keys.
  $unique_ids = array_unique($ids);
  // Use array_intersect_key to filter the original $rows array
  // based on the keys of the unique ids values.
  $unique_rows = array_intersect_key($rows, $unique_ids);
  $rows = $unique_rows;

another code can't pass phpstan exemple
you need to use injection dependency. but why does it always have to be node? there are many other entities like user, taxonomy or custom entities ECK?


    $entity_type_manager = \Drupal::entityTypeManager();
    $revision_ids = \Drupal::entityQuery('node')
....
🇫🇷France lazzyvn paris

Your patch cannot be merged in dev because it does not respect drupal phpstan and phpcs standards

🇫🇷France lazzyvn paris

You can try module field states transition it is 2 times better than content moderate state and kanban is full support module field states transitions

🇫🇷France lazzyvn paris

it can't pass phpcs test

FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
------------------------------------------------------------------------------------------------------------------------
 475 | WARNING | Only string literals should be passed to t() where possible
     |         | (Drupal.Semantics.FunctionT.NotLiteralString)
 509 | WARNING | Only string literals should be passed to t() where possible
     |         | (Drupal.Semantics.FunctionT.NotLiteralString)
🇫🇷France lazzyvn paris

Address Autocomplete has never released an official version, this is not the main reason for me to write a new module. But the configuration of the two modules is different. Address Autocomplete is configured once in the module, while Address Suggestion module is configured in each field. This difference is very big and cannot be merged.

🇫🇷France lazzyvn paris

I knew it. it write down in \datafield\src\Plugin\Field\FieldType\DataFieldItem.php

    // Reset the field storage config property - it will be recalculated
    // when accessed via the property definition getter.
    // @see Drupal\field\Entity\FieldStorageConfig::getPropertyDefinitions()
    // If we don't do this, an exception is thrown during the table update that
    // is very difficult to recover from since the original field tables have
    // already been removed at that point.
    $field_storage_config = $form_state->getBuildInfo()['callback_object']->getEntity();
    $field_storage_config->set('propertyDefinitions', NULL);

You will see the same error on the module having a dynamic subfield like double field, triple field...

🇫🇷France lazzyvn paris

Actually, from beginning, I did not intend to support the smart date module because its data storage structure is completely different from date time and date rage module in drupal core. After supporting smart date, a series of timezone problems appeared, so in the new version, I was forced to prioritize standard drupal fields than supporting smartdate. You can go back last version or wait new version

🇫🇷France lazzyvn paris
Drupal\subpathauto\PathProcessor::__construct(): Argument #4 ($module_handler) must be of type ?Drupal\Core\Extension\ModuleHandlerInterface, Drupal\Core\Path\PathMatcher given in Drupal\subpathauto\PathProcessor->__construct() (line 98 of D:\www\drupal\web\modules\contrib\subpathauto\src\PathProcessor.php).

drupal 11.2
it's weird i get error everywhere

🇫🇷France lazzyvn paris

It must support Bootstrap 5 admin(there is an option to enable Bootstrap 5 if your theme is not based on Bootstrap 5), I user lib bootstrap 5 table for search,... I would be happy if the Gin theme were also supported.
Your merge request doesn't pass the phpcs test, so I definitely can't merge your branch

🇫🇷France lazzyvn paris

I will do it when the customer requests. Once a customer requested but in the end they did not pay so I have no intention of developing further.

🇫🇷France lazzyvn paris

glTF is family of GLB, I verified GLB it works well

🇫🇷France lazzyvn paris

@rakesh.gectcr thanks you save my day.
I confirm patch #49 fixed this issue

🇫🇷France lazzyvn paris

Can you specify which eca module? JavaScript error? Log error, etc.

🇫🇷France lazzyvn paris

purge cache + varnish it works fine after update

🇫🇷France lazzyvn paris

check on drupal 10 and 11 class Serializer the construction doesn't need $module_handler

🇫🇷France lazzyvn paris

I think the module can't support automatic entity label because it uses hook_inline_entity_form_table_fields_alter and hook_entity_insert / hook_entity_presave to generate the label after save, this is not the right way, it should use the event lister on the entity you can do it by your custom module.

🇫🇷France lazzyvn paris

No country is a required field, in my case I am using a custom form with ajax so it can be null. adding ?? '' is not a big deal to avoid a falta error.

🇫🇷France lazzyvn paris

I can't merge this branch, there is warning

   119    Call to deprecated method clearCachedDefinitions() of interface  
         Drupal\Core\Asset\LibraryDiscoveryInterface:                     
         in drupal:11.1.0 and is removed from drupal:12.0.0. Use          
         LibraryDiscoveryCollector::clear() instead.  
🇫🇷France lazzyvn paris

Again you must fixed stylint css. If It doesn’t pass I can’t merge like last one. If you don’t know how to do just create patch I will do latter

🇫🇷France lazzyvn paris

I think it can be report to admin theme, with bootstrap 5 admin theme it works fine

🇫🇷France lazzyvn paris

bootstrap 5 table found in formatter not in widget

🇫🇷France lazzyvn paris

Please fix phpcs and stylelint

🇫🇷France lazzyvn paris

it's weird that version 7.x seems to work fine with rocketchat. Because i remember many years ago i tried with drupal 7 and had almost no errors. Or you can add me as a maintainer, I volunteer my time to port code 7.x for support rocketchat

🇫🇷France lazzyvn paris

i dont support gin theme, full support bootstrap 5 admin theme, if you want you can create widget for gin

🇫🇷France lazzyvn paris

Are you trying to create a patch on the development version? I don't think it's necessary to add this method, as I verified on my local host that it works correctly.

🇫🇷France lazzyvn paris

it already exists you convert to bootstrap 5 table or datatable it is available

🇫🇷France lazzyvn paris

I can contribut my time to this interesting project. please add me

🇫🇷France lazzyvn paris

It can work with custom entities. If you installed with module demo you only see node. It is just views so you can add your custom entities and map with worktime entity

🇫🇷France lazzyvn paris

Check dev version. it fixed

🇫🇷France lazzyvn paris

Please support embeddings in elasticsearch this is very important for AI. It is RAG. it will help the AI agents. So it can use with ollama, prospect of being able to chat with AI it will answer more accurately with the data that has been indexed in elasticsearch

🇫🇷France lazzyvn paris

it's not a bug it's a feature. when a line is empty it will check and delete that line. Please note this module is written for the purpose of storing data and sub field is not like the fields you usually use.

🇫🇷France lazzyvn paris

It is field widget / field formatter

🇫🇷France lazzyvn paris

Has anyone reported the bug to the author of iCalcreator? They didn't think the description to support html so when breaking the line, it have to avoid cutting off the tag html.

🇫🇷France lazzyvn paris

@joachim it is Just because you don't use it doesn't mean you don't want others to not use it.
In fact, Old Drupal worked fine in this environment, for me even after changing htaccess #26 it still crashes, I'll log the errors.

🇫🇷France lazzyvn paris

How did you install the module? by composer or zip? if you installed it with composer, it needs to download erusev/parsedown from the vendor (i guess you are missing this library) this library is needed to parse the Markdown startup icon.
By the way, you don't need to go to goto config to update the icons when the startup icon adds new icons, you need to update them manually or you want to change/add keywords for the icons. Then you need to go to config to update

🇫🇷France lazzyvn paris

Try the dev version, I never checked with multilingual

🇫🇷France lazzyvn paris

You can change annother formatter type it wil show 0. btw it fixed in dev version for default string formatter

🇫🇷France lazzyvn paris

you didn't read the module instructions step 4? It must config field settings, widget, formatter before use. If you can't, you can hire an drupal expert who can find the error in your case and send a patch to fix.

🇫🇷France lazzyvn paris

you didn't read the module instructions step 3?

🇫🇷France lazzyvn paris

Clear cache maybe?
tested on my drupal it works fine

🇫🇷France lazzyvn paris

If you have any contributions to the module, please send a patch and I will review and update as soon as possible.
With data fields, the subfields will be dynamic so you cannot know in advance and create full schemas for it, so there is no mainPropertyName set (it returns always null).
Unfortunately, I do not have any knowledge about ECA, it may have changed now, but I tried it before and abandoned it because it was a bit difficult to use. Currently, none of my customers have requested integration with ECA, so I do not think I will spend more time supporting ECA. The automation solutions I use are n8n simple and easy to use. I have supported datafields that can be exported to json on views and on graphql. You see, n8n only needs to map the input data as a json to perform the next tasks.
I am currently researching to create a module n8n for drupal that can do automation in a simple way.

Production build 0.71.5 2024