🇺🇸United States @smustgrave

Account created on 30 June 2015, almost 10 years ago
  • Software Engineer at Mobomo 
#

Merge Requests

More

Recent comments

🇺🇸United States smustgrave

Not in front of my computer does facets happen to have a test we can steal?

🇺🇸United States smustgrave

Was about say didn't know we used BETA release but then saw "symfony/serializer": "~v7.3.0-BETA1", so nvm

lets do it!

🇺🇸United States smustgrave

Well this did break UserPasswordResetTest, so maybe is needed.

Either way IS is incomplete.

🇺🇸United States smustgrave

@acbramley I don't think DI have been blocker for these tickets. @nicxvan may know the ticket where that's being tracked?

Sounds like good novice tasks for a dev day to go back and add DI to these conversions!

1 comments I do want to make, not 100% phpstan and all is running on hook folders yet but will it ding the empty docs for __construct()? Not a blocker just mentioning.

Looking at the MR seems like a good conversion though

🇺🇸United States smustgrave

Will need UX sign off, but the issue summary seems to be void of screenshots to assist the UX team. Tagging for such

Thanks!

🇺🇸United States smustgrave

The issue summary for this one appears incomplete.

🇺🇸United States smustgrave

This seems to still be relevant.

Will need an issue summary and tagging novice as should be a good task for new users.

🇺🇸United States smustgrave

Thank you for creating this issue to improve Drupal.

We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

Thanks!

🇺🇸United States smustgrave

Thank you for creating this issue to improve Drupal.

We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

Thanks!

🇺🇸United States smustgrave

Thank you for creating this issue to improve Drupal.

We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

Thanks!

🇺🇸United States smustgrave

Thank you for creating this issue to improve Drupal.

We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

Thanks!

🇺🇸United States smustgrave

Thank you for creating this issue to improve Drupal.

We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

Thanks!

🇺🇸United States smustgrave

Thank you for creating this issue to improve Drupal.

We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

Thanks!

🇺🇸United States smustgrave

Thank you for creating this issue to improve Drupal.

We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

Thanks!

🇺🇸United States smustgrave

Thank you for creating this issue to improve Drupal.

We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

Thanks!

🇺🇸United States smustgrave

Thank you for sharing your idea for improving Drupal.

We are working to decide if this proposal meets the Criteria for evaluating proposed changes. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or there is no community support. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

Thanks!

🇺🇸United States smustgrave

Thank you for sharing your idea for improving Drupal.

We are working to decide if this proposal meets the Criteria for evaluating proposed changes. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or there is no community support. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

Thanks!

🇺🇸United States smustgrave

Just following up if this is still needed, if no follow up in 3 months could close out.

🇺🇸United States smustgrave

Since there hasn't been a follow up for reasoning to add in a few years going to close out.

Thanks

🇺🇸United States smustgrave

Since there's been no follow up going to close this one out. If still a bug please re-open updating the issue summary with steps to reproduce.

🇺🇸United States smustgrave

If the bug can't be reproduced in core then, to me, sounds like the contrib modules could be the bug.

🇺🇸United States smustgrave

If it’s just dealing with checkboxes should be able to provide test coverage, we got some existing ones as is

🇺🇸United States smustgrave

Before

After

Resolved what threads I could but restoring status

🇺🇸United States smustgrave

Actually see in #6 ckrina gave the thumbs up.

🇺🇸United States smustgrave

Before

After

Does appear to address the failure

Waiting on a maintainer to make sure they're fine with the color change.

🇺🇸United States smustgrave
1) Drupal\Tests\Core\DependencyInjection\Compiler\BackendCompilerPassTest::testProcess
Expectation failed for method name is "hasDefinition" when invoked 1 time
Parameter 0 for invocation Symfony\Component\DependencyInjection\ContainerBuilder::hasDefinition('.fakeService'): bool does not match expected value.
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-'a_valid_default_backend.fakeService'
+'.fakeService'
/builds/issue/drupal-3518917/core/lib/Drupal/Core/DependencyInjection/Compiler/BackendCompilerPass.php:67
/builds/issue/drupal-3518917/core/tests/Drupal/Tests/Core/DependencyInjection/Compiler/BackendCompilerPassTest.php:101
FAILURES!

Shows the test coverage

Summary appears to match the MR and don't see any open questions.

🇺🇸United States smustgrave

New maintainer is working on a D10/11 release

🇺🇸United States smustgrave

A new maintainer was added last week.

🇺🇸United States smustgrave

This is a co-worker of mine and I have the opt in security coverage if that helps.

🇺🇸United States smustgrave

Since there hasn't been a reason to re-open since 2014, going to close out. If still a valid task in D11 lets re-open updating the summary please.

Thanks all!

🇺🇸United States smustgrave

Since we still don't have steps going to close out.

If still a valid bug please re-open providing steps to reproduce

Thanks all

🇺🇸United States smustgrave

Believe this can be closed as outdated, views as the ability to expose the operators so could do that if you want to run differently.

If I'm wrong please re-open

Thanks!

🇺🇸United States smustgrave

Since it's been a few years with no follow up, going to close out. If still a needed feature in core please re-open.

🇺🇸United States smustgrave

Thank you for creating this issue to improve Drupal.

We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

Thanks!

🇺🇸United States smustgrave

Thank you for sharing your idea for improving Drupal.

We are working to decide if this proposal meets the Criteria for evaluating proposed changes. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or there is no community support. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

Thanks!

🇺🇸United States smustgrave

Thank you for sharing your idea for improving Drupal.

We are working to decide if this proposal meets the Criteria for evaluating proposed changes. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or there is no community support. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

Thanks!

🇺🇸United States smustgrave

Thank you for sharing your idea for improving Drupal.

We are working to decide if this proposal meets the Criteria for evaluating proposed changes. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or there is no community support. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

Thanks!

🇺🇸United States smustgrave

Thank you for sharing your idea for improving Drupal.

We are working to decide if this proposal meets the Criteria for evaluating proposed changes. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or there is no community support. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

Thanks!

🇺🇸United States smustgrave

Thank you for creating this issue to improve Drupal.

We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

Thanks!

🇺🇸United States smustgrave

Thank you for creating this issue to improve Drupal.

We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

Thanks!

🇺🇸United States smustgrave

Thank you for creating this issue to improve Drupal.

We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

Thanks!

🇺🇸United States smustgrave

Thank you for creating this issue to improve Drupal.

We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

Thanks!

🇺🇸United States smustgrave

Thank you for creating this issue to improve Drupal.

We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

Thanks!

🇺🇸United States smustgrave

This came up as a daily BSI target

Can someone confirm if this is still an issue in D11?

Also tagging for issue summary update so if moved back to active lets update that as well please

Thanks!

🇺🇸United States smustgrave

Since there hasn't been a follow up going to close out, if still an issue in the latest version of Drupal please re-open providing steps to reproduce

🇺🇸United States smustgrave

Seems like a good improvement for qualify of life, and debugging

🇺🇸United States smustgrave

Maybe a test case should be added to show the problem and why this fix is needed.

🇺🇸United States smustgrave

Try this again lol, additional changes seem fine. Restoring status

🇺🇸United States smustgrave

Issue summary is incomplete

Should include the full summary template, with steps to reproduce

Also all fixes should be in MRs and as a bug will need test coverage.

🇺🇸United States smustgrave

Issue summary is incomplete.

But also need some more investigation, what's being passed here that's null. If caused by custom code then seems the custom code is wrong.

🇺🇸United States smustgrave

Don't see an MR opened but will need test coverage too.

🇺🇸United States smustgrave

We can't add a test to verify the existence of a class?

🇺🇸United States smustgrave

Appears to be random nightwatch failure, re-running is fine

Restoring previous RTBC status.

🇺🇸United States smustgrave

Appears to have pipeline failures in the unit tests.

🇺🇸United States smustgrave

Seems straight forward, we had a brief conversation in slack about any questions haha

LGTM

🇺🇸United States smustgrave

Going to disagree that we should have test coverage added but will leave in review.

🇺🇸United States smustgrave

Should make them appear inline vs stacked.

🇺🇸United States smustgrave

Meant to say you could self RTBC but thanks!

🇺🇸United States smustgrave

Seems like a good refactor.

🇺🇸United States smustgrave

Appears to have pipeline issues. And shouldn't this be done as part of 📌 Move drupal_find_theme_templates and drupal_find_theme_functions to Theme Registry service Needs work , think for this to merge need to convert an instance where this would be needed

🇺🇸United States smustgrave

Sorry can the title/summary be tweaked? Mentions doing a move but seems just removing stuff?

If you are another contributor eager to jump in, please allow the previous poster(s) at least 48 hours to respond to feedback first, so they have the opportunity to finish what they started!

🇺🇸United States smustgrave

This probably could of been a novice task for a new user.

🇺🇸United States smustgrave

Lets do it,

Wasn't 100% sure how to test but I ran the updates locally and ran the ckeditor commands in package.json and seem to function fine.

🇺🇸United States smustgrave

This needs more information please, #10 mentioned needing to know what rule this is breaking

🇺🇸United States smustgrave

Thank you for sharing your idea for improving Drupal.

We are working to decide if this proposal meets the Criteria for evaluating proposed changes. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or there is no community support. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

Thanks!

🇺🇸United States smustgrave

Thank you for sharing your idea for improving Drupal.

We are working to decide if this proposal meets the Criteria for evaluating proposed changes. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or there is no community support. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

Thanks!

🇺🇸United States smustgrave

Thank you for sharing your idea for improving Drupal.

We are working to decide if this proposal meets the Criteria for evaluating proposed changes. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or there is no community support. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

Thanks!

🇺🇸United States smustgrave

Thank you for sharing your idea for improving Drupal.

We are working to decide if this proposal meets the Criteria for evaluating proposed changes. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or there is no community support. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

Thanks!

🇺🇸United States smustgrave

Thank you for sharing your idea for improving Drupal.

We are working to decide if this proposal meets the Criteria for evaluating proposed changes. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or there is no community support. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

Thanks!

🇺🇸United States smustgrave

Thank you for creating this issue to improve Drupal.

We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

🇺🇸United States smustgrave

Thank you for creating this issue to improve Drupal.

We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

🇺🇸United States smustgrave

Thank you for creating this issue to improve Drupal.

We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

🇺🇸United States smustgrave

Thank you for creating this issue to improve Drupal.

We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

🇺🇸United States smustgrave

Thank you for creating this issue to improve Drupal.

We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.

Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

🇺🇸United States smustgrave

This happen in the 3.x branches? Not a hugh fan of just an empty check as it could be masking a larger problem. But really could drop support for 2.x since 9 is EOL

🇺🇸United States smustgrave

posted in #core-development. @dww was a +1

I'm a +1 because I never know when to use final myself

Going to mark it, guess if it gets approved a follow up to convert current finals can be opened.

🇺🇸United States smustgrave

RemovePathKeyTest may be a good example

Essentially the test would be

Load a fieldBlock and verify the key DOES NOT exist

Run updates

Load the same block and verify the key DOES exist.

🇺🇸United States smustgrave

Spoke to @catch and he said the existing CR approach should be fine.

Not sure if this will make 11.2 in time but seems like a good conversion to me, all template_hooks appear to be deprecated.

🇺🇸United States smustgrave

This came up as daily BSI target

Testing in D11 I can confirm still an issue with error InvalidArgumentException: Invalid translation language (ar) specified. in Drupal\Core\Entity\ContentEntityBase->addTranslation() (line 986 of core/lib/Drupal/Core/Entity/ContentEntityBase.php).

Will need an MR for 11.x, last patch appears to have test coverage but wonder if we can extend an existing one without a full new bootstrap?

🇺🇸United States smustgrave

Can we use the full issue summary template please.

🇺🇸United States smustgrave

May need a test to show this bug and make sure it doesn't come back

Production build 0.71.5 2024