Hungary
Account created on 28 September 2003, about 21 years ago
  • Full stack community organizer at Acquia 
#

Merge Requests

More

Recent comments

🇭🇺Hungary Gábor Hojtsy Hungary

Unassigning for now, since Matt does not have time anymore for this unfortunately.

🇭🇺Hungary Gábor Hojtsy Hungary

Are there specifics to update other than the outdated screenshot? :)

🇭🇺Hungary Gábor Hojtsy Hungary

I somewhat share the default content concerns, although I believe it would be nice to have a way to automatically generate a default legal notice to start working with. However the current default content designates "Drupal CMS" as the responsible legal entity and is full of mentions of Drupal being responsible for stuff, so that is a no-go to ship with I think :) (On top of the grammatical issues in the text).

Data protection is of a particularly high priority for the management of the Drupal. The use of the Internet pages of the Drupal is possible without any indication of personal data; however, if a data subject wants to use special enterprise services via our website, processing of personal data could become necessary. If the processing of personal data is necessary and there is no statutory basis for such processing, we generally obtain consent from the data subject.

🇭🇺Hungary Gábor Hojtsy Hungary

I think text format editor, content type, taxonomy, etc. type of a recipe would not appear as a site or a kit, so at best it would be a subtype of a foundational recipe (if recipes want to do subtyping that is). Would be great to run the three types through the initiative meeting and get feedback there. Then move forward to codify this as it would help move with project browser's listing.

🇭🇺Hungary Gábor Hojtsy Hungary

📌 Decide and update recipe naming and classification standards Needs review has a decent three legged proposal for types, let's discuss there if there is anything in the way to adopt that :)

🇭🇺Hungary Gábor Hojtsy Hungary

@carsoncho: I like those three levels, we should codify the type naming for them, so that tools like drupal.org's API endpoint can properly expose them and Project Browser can offer a UI to find them (or hardcode filters for them). What's in the way to codify this?

🇭🇺Hungary Gábor Hojtsy Hungary

Does this mean recipe types are not anymore intended to be documented? :)

🇭🇺Hungary Gábor Hojtsy Hungary

Interface Translation (locale) in core uses the project and version identification feature of Update Status. (Also Upgrade Status and Upgrade Rector do the same as well as probably a bunch of other modules).

🇭🇺Hungary Gábor Hojtsy Hungary

Indeed, good point! I think renaming the existing one's human readable name back to Update Status is a good way forward. Internally it reflects that naming already.

🇭🇺Hungary Gábor Hojtsy Hungary

Update Manager sounds good and is in line with the rest of the core and contrib features in this area. Also with Package Manager.

🇭🇺Hungary Gábor Hojtsy Hungary

Meeting parser fails at the celebrations thread. Need to figure out why, then can save the notes.

🇭🇺Hungary Gábor Hojtsy Hungary

Fixing credit for James. Thanks all!

🇭🇺Hungary Gábor Hojtsy Hungary

Googling for that specific error message it comes up from https://www.drupal.org/forum/support/module-development-and-code-questio... , looks like you either use that custom module or someone very strongly got inspired by that code (copy-pasted it). This would be output when that code is run (since it echos and dies). The container in this case is not yet initiatialized because the error happens in a global context while the codebase is being assembled. Look at your own site codebase to find this code.

🇭🇺Hungary Gábor Hojtsy Hungary

Is this included in the error like this?

No! This is not for you! Your Drupal installation does not support utf8mb4. Have a fruitless look at https://www.drupal.org/docs/7/api/schema-api/data-types/encoding-collati ... and then forget this module."

🇭🇺Hungary Gábor Hojtsy Hungary

gábor hojtsy made their first commit to this issue’s fork.

🇭🇺Hungary Gábor Hojtsy Hungary

Merged the MR to report the fail from drush so errors are surfaced earlier and in a more useful way.

🇭🇺Hungary Gábor Hojtsy Hungary

My comment in #4 still stands that this undoes the reporting that there may be deprecated API uses that cannot be detected because its not enabled, given the hasImplementations() will not find the hook if the module is disabled and thus the patched code will return no errors.

🇭🇺Hungary Gábor Hojtsy Hungary

I added that category.

The module already had the Developer Tools category which I kept.

🇭🇺Hungary Gábor Hojtsy Hungary

The tests pass fine on the simple ignore fix now I think given the core fixes. We can merge that in to stop the bleeding on older core versions too without changes needed to the tests.

🇭🇺Hungary Gábor Hojtsy Hungary

Added some related issues.

🇭🇺Hungary Gábor Hojtsy Hungary

I believe 📌 Twig Filter "spaceless" is deprecated Active is still outstanding but otherwise this is resolved in core and fixes were released in patch versions of core.

🇭🇺Hungary Gábor Hojtsy Hungary

Merged the MR. Will release 4.3.6 with this and a couple other fixes soon.

🇭🇺Hungary Gábor Hojtsy Hungary

The MR should remove the drupal_root from phpstan-drupal config. The other errors I don't believe come from Upgrade Status :)

🇭🇺Hungary Gábor Hojtsy Hungary

gábor hojtsy made their first commit to this issue’s fork.

🇭🇺Hungary Gábor Hojtsy Hungary

You are right, I made Drupal CMS a lot more prominent on the project page, hope this helps :)

🇭🇺Hungary Gábor Hojtsy Hungary

Add Drupal 10 end of life. Expand Drupal 7 end of life info.

🇭🇺Hungary Gábor Hojtsy Hungary

We discussed this with Lauri yesterday as Drupal core product managers. We have no concerns. The added code does not have UI parts other than some error messages which seemed fine. Two modules already integrate well with it and we agree the product needs it for both. Being an API module, we accept the judgement of the framework managers on the APIs.

🇭🇺Hungary Gábor Hojtsy Hungary

I created the Drupal 11 video with some of the above in mind (establish Drupal as a trusted name, highlight wide contribution, etc) and then get onto the benefits of Drupal 11 itself. I did not use a voiceover for simplicity.

I think if we want to have voiceover for this one, we want to have a video with music + voiceover. Is there a good distribution mechanism that is editable for this? Or we'll use one person's copy / computer to receive and edit the translated voiceovers in? Or distribute the music alongside the video for the local associations at least?

🇭🇺Hungary Gábor Hojtsy Hungary

Those cards look great! I use Google for my searches and never seen that :) I agree adding this would be great, and I think @drumm already outlined that the Drupal 10/11 version of events could receive that later on :) The Drupal 7 version of the site is currently on life support.

🇭🇺Hungary Gábor Hojtsy Hungary

Does core do the same? It does not help much if we parse those files, if core does not do it :)

🇭🇺Hungary Gábor Hojtsy Hungary

Neither sample searches provided showed me event cards. How would those look like when they show up in Google?

🇭🇺Hungary Gábor Hojtsy Hungary

Thanks for opening the issues. I neglected to do that when scheduling them.

🇭🇺Hungary Gábor Hojtsy Hungary

The referenced core issue and above that 📌 Fix Twig 3 deprecations Needs review is where we are actually fixing the core issues that cause this, so Upgrade Status should not fix this for now. The plan is a bugfix release of core soon will fix this. Postponing on that for now.

🇭🇺Hungary Gábor Hojtsy Hungary

These were the top errors found in contrib by Project Analysis:

  • 177 765 times (in 1324 projects): Since twig/twig 3.12: Getting node "filter" on a "Twig\Node\Expression\FilterExpression" class is deprecated.
  • 10734 times (in 641 projects): Since twig/twig 3.12: Not passing an instance of "TwigFunction" when creating a "render_var" function of type "Twig\Node\Expression\FunctionExpression" is deprecated.
  • 1080 times (in 209 projects): Since twig/twig 3.12: The "tag" constructor argument of the "Drupal\Core\Template\TwigNodeTrans" class is deprecated and ignored (check which TokenParser class set it to "trans"), the tag is now automatically set by the Parser when needed.
  • 1100 times (in 274 projects): Since twig/twig 3.12: Getting node "filter" on a "Twig\Node\Expression\Filter\RawFilter" class is deprecated.
  • 864 times (in 121 projects): Since twig/twig 3.12: Getting node "filter" on a "Twig\Node\Expression\Filter\DefaultFilter" class is deprecated.
  • 512 times (in 147 projects): Since twig/twig 3.12: Twig Filter "spaceless" is deprecated. See https://drupal.org/node/3071078.

Are these all being resolved in the children? :)

Also these two were found in one project each, not a priority IMHO:

  • Since twig/twig 3.12: Character """ at position 2 should not be escaped; the "\ character is ignored in Twig v3 but will not be in v4. Please remove the extra "\" character." in formatage_models 4.x-dev
  • Since twig/twig 3.12: Character "T" at position 7 should not be escaped; the "\ character is ignored in Twig v3 but will not be in v4. Please remove the extra "\" character." in festeva 1.0.x-dev
🇭🇺Hungary Gábor Hojtsy Hungary

Imo we shouldn't just `fix deprecation message` here. But fix deprecations deeper, with research of new twig extending style. This issue is RTBC of course. But - can we do it better? Idk yet.

I think it would be great to have a "good enough" solution ASAP, and then spend any time to fix it even better later. Because these deprecated API uses produce errors with most runs of Upgrade Status on all contrib projects and people can't do anything about it. When we ran Project Status on all contrib modules, these produced 190 thousand twig deprecacted API calls detected across all of Drupal 10 contrib :) This also means that none of those are getting project update bot patches/MRs because they have these unsolvable issues. So a quick solution would be important to unblock the rest of the ecosystem / tooling, then a better solution is not out of question :)

🇭🇺Hungary Gábor Hojtsy Hungary

Retitled for that scope then. Should we open an MR in an another issue or conver this to a core issue?

🇭🇺Hungary Gábor Hojtsy Hungary

Indeed that error message is not helpful. Could the error be more specific and tell the user the directory does not exist? What can the user then do to get the directory created? (Save the file settings config form?) At least if we let the user know this, they have a chance to sync their translation files if they wish so without silently getting them into a different situation.

🇭🇺Hungary Gábor Hojtsy Hungary

Uh the tests are also getting caught up with the twig problems, they need to be updated to expect them in an "ignore" category.

🇭🇺Hungary Gábor Hojtsy Hungary

We also noticed 190k or so of these errors with twig 3.12 across scanning all of contrib with Deprecation Status. Since the error (almost?) always come from core not being compatible, we should not burden users with trying to figure this out. I proposed an MR here that puts the error in the "ignore" category. It is not required to resolve twig 3.12 incompatibilities anyway to be compatible with Drupal 11 :)

🇭🇺Hungary Gábor Hojtsy Hungary

Ah right, good point. I would try composer why-not https://laravel-code.tips/use-composer-why-not-to-see-why-a-dependency-c... on symfony/css-selector ^4.4

🇭🇺Hungary Gábor Hojtsy Hungary

How is your problem related to Upgrade Status?

🇭🇺Hungary Gábor Hojtsy Hungary

Thanks for starting on this! "Within the core committer team", "while core committers", etc. still has the old terminology.

🇭🇺Hungary Gábor Hojtsy Hungary

So the text is not in line with the requirement?

    elseif ($database_type == 'sqlite') {
      $database_type_full_name = 'SQLite';
      $requirement = $this->t('When using SQLite, minimum version is 3.26');
      if (version_compare($version, '3.45') >= 0) {
        $class = 'color-success';
      }
      else {
        $status = FALSE;
        $class = 'color-error';
      }
    }

I see it here.

🇭🇺Hungary Gábor Hojtsy Hungary

Moving the localization server which is where the error shows up. Can you not save it or "just" an error shows up but you can still save it?

🇭🇺Hungary Gábor Hojtsy Hungary

How is this going? Moving to Drupal CMS.

Production build 0.71.5 2024