🇫🇷France @matoeil

Account created on 18 December 2015, over 8 years ago
#

Recent comments

🇫🇷France matoeil

found that
<lots><value>lot 1</value><value>lot 2</value></lots>
work if the parser is configured as parse subdirectories with a matching /lots/value

The module is lacking clearly of documentation or hint in the back office or infos using the tour module

🇫🇷France matoeil

what if it s not there ?
i cannot see it ( see capture)

🇫🇷France matoeil

i do use the database backend for searching, and i cannot see that "partial matching option".

Could u tell me the url and screen capture ?

🇫🇷France matoeil

i don't see the relevance of that relevance sorting. Who would want an unrelevant sorting ?

🇫🇷France matoeil

is it safe to put lax then regarding CSRF vulnerabilities ?

🇫🇷France matoeil

the parameters was not there in the services.yml
it should be lax by default.

Anyway , adding

parameters:
  session.storage.options:
     # Set the SameSite cookie attribute: 'None', 'Lax', or 'Strict'. If set,
     # this value will override the server value. See
     # https://www.php.net/manual/en/session.security.ini.php for more
     # information.
     # @default no value
     cookie_samesite: Lax

seems indeed to fix it

🇫🇷France matoeil

i have looked more thoroughly on our use case.
We do use a lot of entity ref fields and i can spot at least 2 problems

if having a content type let's say 'page' , having a paragraph entity ref field, having a view ref field displaying a list of content, let s say 'articles' with their respective fields.
when using front editing here to edit the article, the article is revisioned, but the page is not.

if having a content type , lets say 'article', having a media ref field
when using front editing here to update the media, the article is not revisioned

I am editing the issue

🇫🇷France matoeil

to precise #3:
if using back-office editing , a revision is created each time a modification is made
if using front end editing , a revision is, in certain cases, not created.

If someone revert a revision, all modif that would have been done without revision are lost.
That is why i have considered it critical.

🇫🇷France matoeil

using dev version ( cf#115) and patch #149 :
it work well when the visibility for the field is conditionned by a checked boolean or a value or a list field.

It does not work when the control field is a taxonomy entity ref field.

🇫🇷France matoeil

nice, but it's not enough.
It won't work on sites using a base theme different than "stable".

🇫🇷France matoeil

adding the links to olivero solve the problem as long as content translation is not used

function child_preprocess_links(&$variables) {
  if (!empty($variables['links'])) {
    foreach ($variables['links'] as $key => $value) {
      if (!is_numeric($key)) {
        $class = Html::getClass($key);
        $variables['links'][$key]['attributes']->addClass($class);
      }
    }
  }

When trying to edit paragraphs on a fr or en version of one content , the modifications affect both versions ( 1revision for each language is created)
Are these links language aware ?

🇫🇷France matoeil

-adding content translation
- configure the paragrph to be translatable
-adding to the basic page a paragraph ref field
- create a content in default language
- translate

=> the edited paragraph is saved on both languages.
1 revision is made for each language

the edition in one language should not affect the other language !

are the geysir links language aware ?

🇫🇷France matoeil

it solves it with the preprocess_links hook

🇫🇷France matoeil

i am in php7.4 yes. I guess u can close the issue.

🇫🇷France matoeil

does not work for me with umami distribution

🇫🇷France matoeil

as the Rendered HTML output solves it for me, u can choose to close the issue.
This feature would be useful though when not wanting to index the whole content of the entity

🇫🇷France matoeil

There is a /modules/contrib/n1ed/src/Plugin/CKEditorPlugin/N1ED.php file

🇫🇷France matoeil

The problem is the module leave to content editors of the site, that are not Drupal experts, to choose one option or the other, from one media to another, without understanding fully the consequences of it ( and without knowing how the site has been configured )

If the use of this module was to always overwrite the file , it would be better then to be configurable globally, and not left to the choice of the content editor on every media

🇫🇷France matoeil

Thnaks for taking that time to reply.
I understand your point, but why absolutely wanting to preserve drupal core's behaviour when it is not working properly ?
That module could fill a Drupal flaw.

Unless you do settings that are not recommended in production, that sentence is wrong :
"If unchecked, the filename of the replacement file will be used, and the original file may be deleted if no previous revision references it"

The original file is never deleted !

In other words, for sites in production, the media entity file replace module does not work as it says it should.

🇫🇷France matoeil

As previously said on #7 though:

- this module does not work as it should at the moment
- when doing replacement , the old file is never deleted , which causes data protection issues
- unless ['make_unused_managed_files_temporary'] = TRUE is set, which is highly not recommended because drupal core has issues with unmanaged files
-these core bugs have been on for 6years so it might not been fixed exactly tomorrow.

may i leave your reconsider as deleting the file immediately would fix the problem?

🇫🇷France matoeil

An example of nested fields to add when using nested paragraphs

🇫🇷France matoeil

If the current date is in the range of a seasonal weektable then display that one.

If the current date is less than 7 days of a change of timetable, a message could be giving that information

🇫🇷France matoeil

my mistake, it does not work actually

🇫🇷France matoeil

this seems to work :

function my_module_theme($existing, $type, $theme, $path) {
  return [
    'my_module_theme' => [
      'template' => 'my_module',
      'variables' => [
        'events' => my_function(),
        'count' => my_other_function(),
      ],
      'cache' => false,
    ],
  ];
}

it is not indicated as possible parameter in the doc

https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Render%21...

🇫🇷France matoeil

I understand it is not possible to import file to media ref fields ?
https://www.drupal.org/project/feeds/issues/2928904 Add a mapping target to media field Needs work

It would be useful to indicate that limitation explicitely on the /project/feeds page.

🇫🇷France matoeil

Is there an example of what information must contain the xml xpath to migrate a file from D7 to a D9 file media, using feeds?

In my case above , once correcting the text size, a media is created but it is not linked to the file system.

🇫🇷France matoeil

The documentation should make it explicit that it is a 2 steps thing:

create a feed type on /admin/structure/feeds

create a feed on /admin/content/feed , and this is when the source is indicated.

Production build 0.69.0 2024