🇧🇷Brazil @jkamizato

I've created an issue in Masterminds/HTML5, asking them to upcast the function `fwrite` , then it fixes the error:

https://github.com/Masterminds/html5-php/issues/254

However, the problem is that this plugin is dependent on Drupal core. I am not sure if it's a core issue or $dom->nodeValue null has to be fixed here.

jkamizato → created an issue.

The phpunit test was fixed and I'm attaching the patch.

I know it needs to fix phpunit issue, but here the latest patch version with phpstan and phpcs fixes

I've created a Merge Request and the patch. Also changed to "Needs Review"

jkamizato → created an issue.

@eiriksm Merge Request created

Update:
Fix "Undefined variable $authUsername" error

Changed status to "Needs Review"

Sorry, wrong patch file. I'm updating the correct file.

Hi @solideogloria, thanks for your advice.

I've recreated this patch using key module dependecy ( https://www.drupal.org/project/key → )

Now, admin user can create a new key entry (user/pass) and reference it in linkchecker configuration.

Hi everyone,

I've created simplest approach, skipping new fields, and configuration. It's passing basic authorization inline, like "login:pass@host.com" in Base Path field.

jkamizato → created an issue.

jkamizato → created an issue.

Thanks @salmonek! Patch in #5 fixed the issue.

jkamizato → created an issue.

Hey guys, this works for me and I've created a patch file while new version is not released.

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

I've opened a bug in Ckeditor Premium Feture module as well, in case its their issue: https://www.drupal.org/project/ckeditor5_premium_features/issues/3373997 🐛 When Track Changes is on, Link style module does't work Needs work

jkamizato → created an issue.

jkamizato → created an issue.

Hey here the patch for 1.2.x branch

jkamizato → created an issue.

Replace ckeditor5_font to  https://www.drupal.org/project/ckeditor_font → , disable editor_advanced_link if you are using and disable Js aggregation.

I'll check if there is a way to disable aggregation only for ckeditor.

Guys, I've created a new branch 3350254-ckeditor-5-36-2 and I've applied manually the patch #23. I'm uploading the new patch and I've updated the code.

I've moved to "need to work" because MR and patches can't be applied.

Tested, good job @icurk !!!

I've changed to priority major, because without this patch Drupal 10 broke.

jkamizato → created an issue.

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

In case someone need the patch file

jkamizato → created an issue.

Hey team, I've pushed a update for Drupal 10. Hope its all ok. Thanks!!!

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

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

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

I've change the approach. It was easier to user select option list instead of popup.