Ontario, CA 🇨🇦
Account created on 22 April 2009, about 16 years ago
#

Merge Requests

More

Recent comments

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

I don't see how this is outdated; it's still something that could be useful. If you don't want to have this change, the status could be "won't fix".

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

liam morland made their first commit to this issue’s fork.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

liam morland made their first commit to this issue’s fork.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

liam morland made their first commit to this issue’s fork.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

liam morland made their first commit to this issue’s fork.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

liam morland made their first commit to this issue’s fork.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

liam morland made their first commit to this issue’s fork.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

liam morland made their first commit to this issue’s fork.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

Looks good.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

Thanks for the patch.

Why is this change required? Please update the issue summary.

Tests are not passing.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

Looks good except instead of "This ensures the NameAdditionalPreferredTrait is used correctly", I suggest "This is used by NameAdditionalPreferredTrait" or something like that.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

Thanks for the patch.

Please make coding standards fixes.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

Looks good

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

There could be a 2.0 version of http_status_code which is based on redirect but adds non-redirects like 410.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

Drupal 7 is no longer supported.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

This is still applicable to the Drupal 8+ version.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

The merge requests should be updated to target branch 2.0.x.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

Closing based on #5.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

The issue may be that it is using the "Automatically include table of contents" setting instead of the "Display table of contents in a block" setting. I have implemented this change in the merge request.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

liam morland created an issue.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

The merge request currently makes a change and then un-does it.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

liam morland made their first commit to this issue’s fork.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

The merge request avoids the issue in #12 and it has a test.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

Unless I am misunderstanding something, this should be closed as duplicate of 🐛 Fatal error after updating to toc_api:^2 Active .

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

Let's keep the discussion about that patch in that issue.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

What is the reason for using dependency injection and non-static methods for this? This way it will have to persist an instance of DeleteUsersBatch in the batch process. I don't think that is an improvement.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

These changes are included in the merge request in 📌 Update for Drupal 10/11 Active .

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

I don't understand. The comments in 🐛 Layout builder fatal error Active say to apply the patch in this issue.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

What has to be released? The latest toc_filter 8.x-2.x is tagged and released as 8.x-2.3.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

What is the patch that needs to be applied?

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

liam morland created an issue.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦
🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

I haven't worked on merge request 6.

Composer files shouldn't refer to patches like: https://git.drupalcode.org/project/layout_builder_st/-/merge_requests/5....

These change anytime someone makes a commit so you can't be sure you'll get the same code each time you build.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

liam morland created an issue.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

liam morland made their first commit to this issue’s fork.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

In 4.x, these were fixed in commit 83b89c8. Please switch the merge request back to targeting 3.x.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

Drupal 7 is no longer supported. If this issue applies to a supported version, please re-open.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

Drupal 7 is no longer supported. If this issue applies to a supported version, please re-open.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦
🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

Drupal 7 is no longer supported. If this issue applies to a supported version, please re-open.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

Drupal 7 is no longer supported. If this issue applies to a supported version, please re-open.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

Drupal 7 is no longer supported. If this issue applies to a supported version, please re-open.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

liam morland made their first commit to this issue’s fork.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

Do we not still need the fix in the merge request?

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

liam morland made their first commit to this issue’s fork.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

Re-rolled. The same test is failing. Perhaps a better test would be to check the favicon element on the page.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

Thanks. But the way there is no need to make a new merge request. The new code can be force-pushed onto the same branch and the target branch of the merge request can be changed.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

Please put your change into a merge request.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

@haritha-c Thanks for the patch. The merge request needs to target 8.x-1.x. When it is ready, please set the issue to "Needs review". Someone else can then test and review, leading to it being set "Reviewed & tested by the community".

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

All coding standards checks now pass.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

liam morland created an issue.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

Unless I am missing something ::getLastDelimitorBehaviors() gets renamed to ::getLastDelimiterBehaviors(). In the case of ::getLastDelimitorTypes(), it's still there but issues a deprecation notice and then calls ::getLastDelimiterTypes().

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

This should be active until there is a patch to review or work on.

@belba, this is assigned to you. Do you intend to work on it? If not, this should not be assigned to you.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

This change was committed in 💬 Drupal 10 compatibility Fixed .

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

There needs to be a deprecation for getLastDelimitorBehaviors() as well.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

I think it would be reasonable to fix just the regression and then move on to 4.1.x.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

All changes have been made in 📌 Make composer.json match info file Active .

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

Thanks

By the way, you could shorten your SettingsForm class by using constructor property promotion.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

liam morland made their first commit to this issue’s fork.

🇨🇦Canada Liam Morland Ontario, CA 🇨🇦

If changes like this are going to be made, it makes it very urgent to get a branch which is aimed at currently-supported versions of Drupal. This change introduces a coding standards error. Contributing to this module will become more difficult because people will need to remember to write in an old version of PHP. remembering what changes have been made and what coding standard issues to ignore.

Production build 0.71.5 2024