🇺🇦Ukraine @AstonVictor

HI there,

seems like it works in 2.0.x version.

Steps for testing:
1. install image_url_formatter 2.0.0 & image_widget_crop 3.0.0 modules.
2. create a new image style that uses the manual crop effect.
3. configure the image field to use the crop widget on the manage form display page.
4. configure the field to use the image URL formatter on the manage display page.
4. create a new node with an image (+ crop image) and go to the node overview page.

As a result, I can see the ...?h=d5171e05&itok=Vt7zgUHe suffix.

astonvictor → made their first commit to this issue’s fork.

Hi there,

Seems like you are using an incorrect formatter. The "Image Wrapped Within Link Field" formatter comes from the Image Link Formatter → module.

The current module only renders the plain text without any links. So, you cannot add e.g. target attribute to the plain text.

thanks in advance

Hi there,

I tried to reproduce it, but it works as expected. Moreover, the module template uses {{- var -}} filter to trim empty spaces.

I can only see empty spaces if Twig debug is enabled in the services.yml file. In that case, spaces will be added to all fields. So, it's not related to the current module.

thanks in advance

Hi there,

The config schema is used to validate config values. To check it, you can use the Configuration Inspector → module (attached a screenshot with check results).

thanks in advance

astonvictor → made their first commit to this issue’s fork.

astonvictor → created an issue.

Hi there,

I implemented a simple MR to hide the system name column and add the name after the description.
✨ Provide a simple/lightweight/clean mode that's primarily only the filter Active issue can be related. It helps to show/hide some sections on the page.

thanks in advance

Hi there,

It should be possible to configure sameSite parameter via the services.yml file. It has the 'Lax' value by default in the core.services.yml file.
You can add your own value e.g.

parameters:
  session.storage.options:
    cookie_samesite: Lax

astonvictor → made their first commit to this issue’s fork.

Hi there,

Created a simple config form (with a config schema, menu links, etc) to disable some sections on the page.
Thanks in advance

astonvictor → made their first commit to this issue’s fork.

Hi there,

Updated the list of items to filter on.
Please, check if it works for you

Thanks in advance.

As I can see, it was fixed in 4.x version.

As I can see, it was fixed in 4.x version.

+1 RTBC

Hi there,

I can see the Uncaught TypeError: Cannot set properties of undefined (setting 'attach') console error on Drupal 10.4 on the /admin/people/permissions page.

The patch with updating the dependency to the correct name works for me.
Thanks in advance.

Hi there,

Implemented a simple update hook to force enable the module.
Thanks in advance

astonvictor → made their first commit to this issue’s fork.

astonvictor → made their first commit to this issue’s fork.

I can confirm the patch works for me.

astonvictor → created an issue.

works for me, merged to the dev branch
thanks a lot

Thanks for opening the issue and preparing a new MR.

> In fact, no matter what I chose, it would always render as an h1. Perhaps I'm missing something?
Please, check ✨ Allow node/entity to display title/label field as normal Needs work issue. Th title wrapper is not working for /node/{nid} page.

as for the p wrapper. seems, it's pretty simple MR. so, +1 to review

thanks, merged

astonvictor → created an issue.

refs to steps

I had an error when trying to replace 404 response with 403 response for some cases.
e.g. when a user tries to access /user/2 page, but user 2 was removed.

to replace the response, I used an event subscriber with $events[KernelEvents::EXCEPTION] event
with a callback smth like :

if ($event->getThrowable() instanceof NotFoundHttpException && 'user/2' == $path) {
  $event->setThrowable(new AccessDeniedHttpException());
}

works for me

thanks in advance

fixed in a new 4.2.x branch
thanks

astonvictor → made their first commit to this issue’s fork.

it's already fixed and merged.

fixed in a new 4.2.x branch
thanks

fixed in a new 4.2.x branch
thanks

fixed in a new 4.2.x branch
thanks

fixed in a new 4.2.x branch
thanks

fixed in a new 4.2.x branch. it will automatically check the value and set the type based on it.
thanks

fixed in a new 4.2.x branch
thanks

fixed in a new 4.2.x branch
thanks

fixed in a new 4.2.x branch
thanks

fixed in a new 4.2.x branch
thanks

fixed in a new 4.2.x branch
thanks

thanks

astonvictor → created an issue.

Hi there,

I guess it's related to the 'Views Bulk Operations' module. I can reproduce the same issue with the default operations.
Steps to reproduce:
1. generate 50 nodes.
2. create a new view that shows nodes with VBO and a pager (20 items per page).
3. go to the view and execute e.g. 'publish content' operation -> should work ok.
4. disable the JS in the browser.
4. execute e.g. 'unpublish content' -> doesn't work / there is still some published content.

thanks

astonvictor → made their first commit to this issue’s fork.

As a temporary solution for my project, I removed the column.
thanks in advance

astonvictor → created an issue.

astonvictor → created an issue.

just FYI

> @astonvictor what is the difference between the old merge request and the one you just opened? Which one does your RTBC apply to?

I was not able to apply MR#63 to my project because it conflicts with the patch from ✨ Make random strings optional, to allow linking to footnotes. Active
Both MRs update the FootnotesCkeditorPluginTest.php file

thanks

+1 RTBTC

astonvictor → made their first commit to this issue’s fork.

it works for me.
thanks a lot

basically, I need an option/event to check when the tfa status (1/0) has been changed.

Added a simple patch for now.
I guess it should be updated to use DI.

astonvictor → created an issue.

Hi there,

I tried different themes and was not able to reproduce it and I don't see any span elements in vbo_export module.
Maybe it's smth related to the views_bulk_operations → module that creates a config form for operations or smth related to your admin theme.

thanks in advance

I guess it was done by https://www.drupal.org/project/projectownership → project.

If you want, you can restrict my permissions e.g., delete permission to create new releases, etc.

Hi there,

I didn't work on 7.x version. I only worked on 3.x version.
Here is the list of tasks I worked on - https://www.drupal.org/u/astonvictor/issue-credits/710994 →

thanks

Hi there,

I prepared a new MR with the fix.
thanks in advance

Hi there,

I prepared a new MR with the fixes for merge conflicts.
thanks in advance.

astonvictor → made their first commit to this issue’s fork.

astonvictor → created an issue.

Hi there,

I prepared a new MR that changes the export and updates tests.
thanks in advance

astonvictor → made their first commit to this issue’s fork.

Hi there,

Prepared a new MR with a plugin to export data as a doc file. For exporting data, it uses phpoffice/phpword package.

Please, check if it works for you.

p.s. I didn't update the requirements hook because the package doesn't have a changelog file from the 1.2.0 version. And there is no sense to use 1.1.0 version because it uses the deprecated utf8_encode() function.
thanks in advance

astonvictor → made their first commit to this issue’s fork.

Hi there,

Please, check if the MR works for you.
thanks in advance

Hi there,

I prepared a new MR with the fix. Tested for both header and row.

astonvictor → made their first commit to this issue’s fork.

astonvictor → changed the visibility of the branch 3504860-upgrade-phpofficephpspreadsheet-to to hidden.

astonvictor → made their first commit to this issue’s fork.

Hi there,

I prepared a new MR to support a new version.

Tested with PhpSpreadsheet library: Installed version: 2025-02-07 - 3.9.1. version (info from the status report page) and it works fine for me.

thanks

astonvictor → made their first commit to this issue’s fork.