🇧🇾Belarus @alexdoma

any updates?

@nod_ @quietone
@anjaliprasannan Thank you for your feedback. But i think you have provided incorrect example.

Upon encountering this error for the first time, I also came to the conclusion that using HtmlCommand would be the best solution. However, I have a large number of places in my code where I use ReplaceCommand

The desire to get rid of the jQuery library is commendable, but not at any cost.

Frankly, I don’t understand why I should rewrite my code because of a bug in the core. I haven’t seen any updates in the documentation for ReplaceCommand indicating that it doesn't work for certain tags.

I believe the parent issue (#3238870 Refactor (if feasible) use of jquery parseHTML function to use vanillaJS) was resolved incorrectly and it needs to be reopened. In the meantime, the changes should be reverted.

Replace one cell not working.

// Not working.
$ajax_response->addCommand(new ReplaceCommand($selector, '<td>Hello!</td>')); 

Replace one table working.

// Working.
$ajax_response->addCommand(new ReplaceCommand($selector, '<table><tr><td>Hello!</td></tr></table>'));

PLease, read whole message i have wrote my test case 2 times.
I have provided correct test case with an error.

I don`t want to replace full table like in your case. I want to replace one cell.
Case for one cell is not working.

P.S. in your test case you missed closing </tr> thats why you have 2 rows.

@anjaliprasannan
i have checked your example and get the same results as you.

But, in your html output additional '

alexdoma → created an issue.

@codebymikey

after your last changes i got an error
Fatal errors. The uploaded file size may have exceeded the maximum file size (100 MB) that this server supports.

Patch #159 working for me correctly

update patch to changes in 1.5 version

alexdoma → created an issue.

I got an error message after adding this patch
TypeError: Drupal\flag\Plugin\ActionLink\AJAXactionLink::__construct(): Argument #5 ($csrfToken) must be of type Drupal\Core\Access\CsrfTokenGenerator, Symfony\Component\HttpFoundation\Request given

patch from #14 updated

@berdir thank you. You are right, i have used patch form this one - https://www.drupal.org/project/flag/issues/3168014 🐛 TypeError: Argument 1 passed to Drupal\flag\TwigExtension\FlagCount::count() must implement interface RTBC

update previous patch, fix folder name

alexdoma → created an issue.

Confirm that patch #2 works for me.

add some validations

Update latest patch. Added validations when nothing selected.

Needs work with tests and needs review

patch #6 now working for me
i have get code from https://www.drupal.org/project/hide_revision_field → module and its helps for me
drupal 10.1.3

add full patch

features files was removed here - https://www.drupal.org/project/drupal/issues/3261245#comment-14539406 →
re-roll for drupal 10.1.0

re roll for drupal 10.1.3

@admirlju
i got an error on drupal 9.5
TypeError: Drupal\business_rules\EventSubscriber\KernelRequestListener::onKernelRequest(): Argument #1 ($event) must be of type Symfony\Contracts\EventDispatcher\Event, Symfony\Component\HttpKernel\Event\RequestEvent given in Drupal\business_rules\EventSubscriber\KernelRequestListener->onKernelRequest() (line 37 of modules/contrib/business_rules/src/EventSubscriber/KernelRequestListener.php).

re-roll patch

alexdoma → created an issue.

comment #19 helps me - resaving the widget in the manage form display

set correct paths, re-roll for drupal 9.5.x

fix previous patch - set correct paths, re-roll for drupal 9.5.9

re-roll for drupal 9.5.9

small fix, set correct path

alexdoma → created an issue.

Update previous patch due to php 8.1 Deprecated function: substr_count()
Deprecated function: substr_count(): Passing null to parameter #1 ($haystack) of type string is deprecated in Drupal\entity_reference_revisions\Plugin\Field\FieldFormatter\EntityReferenceRevisionsEntityFormatter->viewElements() (line 144 of /var/www/html/web/modules/contrib/entity_reference_revisions/src/Plugin/Field/FieldFormatter/EntityReferenceRevisionsEntityFormatter.php)

alexdoma → created an issue.

re-roll for 9.5.x

re roll the changes from comment #92