🇺🇸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

So really just need 1 MR pointed at 11.x as that's the development branch.

🇺🇸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

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

If this is still needed can we update summary/title?

If no follow up we could close out in 3+ months.

🇺🇸United States smustgrave

@catch following up on what follow up you were referring to 6 years ago?

🇺🇸United States smustgrave

#1978714: Entity reference doesn't update its field settings when referenced entity bundles are deleted was the issue this code was added to, but don't see anything in that ticket about this being a security issue. So can the issue where this was a security issue? If no follow up I say we close out.

🇺🇸United States smustgrave

Hello this came up as a daily BSI target, since it's been a few years wonder if still experiencing this in D11?

🇺🇸United States smustgrave

Also tried testing on a standard profile install and could not reproduce.

Since it's been 3+ months since #9 without a follow up going to close out, but if still experiencing this in D11 please reopen.

Thanks all

🇺🇸United States smustgrave

Been about a month without additional review. Will RTBC but leaving the one thread open.

🇺🇸United States smustgrave

Seems straight forward, searched for generateAbsoluteString and generateString and all replaced.

🇺🇸United States smustgrave

Seems straight forward, been told since navigation is experimental then a CR isn't needed. Also believe this qualifies for not needing test coverage

🇺🇸United States smustgrave

Was wondering what for XB this was impacting haha :)

🇺🇸United States smustgrave

Appears to have a pipeline issue. Would you say the 2 threads are good to resolve? If they are and you can't I can do it for you.

🇺🇸United States smustgrave

Hiding patches and closing old MRs

Following the steps I get a request like

https://drupal-11x.ddev.site/views/ajax?status=All&title=&type=All&_wrapper_format=drupal_ajax&title=&type=All&status=All&view_name=content&view_display_id=page_1&view_args=&view_path=%2Fadmin%2Fcontent&view_base_path=admin%2Fcontent&view_dom_id=163f73bb84d4e170f0b939de25d6c337ec0daf0d5e472897f73c099a6e1eb94f&pager_element=0&_drupal_ajax=1&ajax_page_state%5Btheme%5D=claro&ajax_page_state%5Btheme_token%5D=U8_NBi2POrE9U1ELs8h7ziadng1Svy8k__iIvXtLat0&ajax_page_state%5Blibraries%5D=eJx9j20OwjAIhi_UbWfwJIZ2ZKuy0gCdH6e36hqjS_zTwMMDeevjdMwx4-C3wgUC4WGUkoH6BGtn4HWjE7EH6tRuFNPkAifDqxWg5n9IV42z_leMmTxIlQTbGILFFV_bX7ymIJwRRpQ9F9TMSevifqZIGOzNE8sCFO_odGaxUKyprXdbpqFl--l71AAZD-MSkyuK0g486z7Wz6lbI150eL09nOD6BRYeC-EDDheR7w

After applying the MR

I get

https://drupal-11x.ddev.site/views/ajax?_wrapper_format=drupal_ajax&title=&type=All&status=All&view_name=content&view_display_id=page_1&view_args=&view_path=%2Fadmin%2Fcontent&view_base_path=admin%2Fcontent&view_dom_id=6bd75558a30df1be7b7f74ebc9fc473a68a2dc3c5e496a831679f7aed2d7155a&pager_element=0&_drupal_ajax=1&ajax_page_state%5Btheme%5D=claro&ajax_page_state%5Btheme_token%5D=U8_NBi2POrE9U1ELs8h7ziadng1Svy8k__iIvXtLat0&ajax_page_state%5Blibraries%5D=eJx9j20OwjAIhi_UbWfwJIZ2ZKuy0gCdH6e36hqjS_zTwMMDeevjdMwx4-C3wgUC4WGUkoH6BGtn4HWjE7EH6tRuFNPkAifDqxWg5n9IV42z_leMmTxIlQTbGILFFV_bX7ymIJwRRpQ9F9TMSevifqZIGOzNE8sCFO_odGaxUKyprXdbpqFl--l71AAZD-MSkyuK0g486z7Wz6lbI150eL09nOD6BRYeC-EDDheR7w

Not the best with nightwatch review and test-only pipeline doesn't work with nightwatch But

browser.assert.strictEqual(
              url.searchParams.getAll('title').length,
              1,
              'Duplicate title parameter',
            );

Should show the duplicates so believe it's valid coverage

Going to mark.

🇺🇸United States smustgrave
1) Drupal\Tests\system\Functional\System\StatusProfileWithoutVersionTest::testStatusPage
Behat\Mink\Exception\ExpectationException: Current response status code is 500, but 200 expected.
/builds/issue/drupal-3511868/vendor/behat/mink/src/WebAssert.php:888
/builds/issue/drupal-3511868/vendor/behat/mink/src/WebAssert.php:145
/builds/issue/drupal-3511868/core/modules/system/tests/src/Functional/System/StatusProfileWithoutVersionTest.php:46

Shows the test coverage

Manually tested following the steps
Deleted version from minimal
Destroyed my local ddev and rebuilt
Visited admin/reports/status and verified the error
Applied the MR and it fixed the issue

LGTM

🇺🇸United States smustgrave

Looking at the steps I would assume a test could be written.

🇺🇸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

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

So just following up if this should be closed?

🇺🇸United States smustgrave

Actually going to close this one out as there hasn't been a follow up originally in 5 years. If still a valid task please re-open

🇺🇸United States smustgrave

Looking at #14 again and believe this is a duplicate so will close out. If I'm wrong please reopen.

🇺🇸United States smustgrave

RTBC as 10.5 looks good.

Know this is a bug but do we push version updates to releases?

🇺🇸United States smustgrave

Since there hasn't been a follow up going to close out, if still an issue in D11 please reopen

Thanks all!

🇺🇸United States smustgrave

Came up as a daily BSI target

Can anyone confirm this is still an issue in D11?

🇺🇸United States smustgrave

Opened 🌱 META: Use dependency injections in permission callbacks Active to track others

Change here though seems pretty straightforward though, good deprecation.

LGTM

🇺🇸United States smustgrave

Not 100% sure putting into review will grab the attention.

Though it's good to have the hard part done in a test.

Will post in slack though to try and kick start it.

🇺🇸United States smustgrave
1) Drupal\KernelTests\Components\ComponentRenderTest::testRender
Drupal\Core\Render\Component\Exception\InvalidComponentDataException: Unable to render component "sdc_test:my-banner". A render array or a scalar is expected for the slot "banner_body" when using the render element with the "#slots" property
/builds/issue/drupal-3504477/core/lib/Drupal/Core/Render/Element/ComponentElement.php:118
/builds/issue/drupal-3504477/core/lib/Drupal/Core/Render/Element/ComponentElement.php:65
/builds/issue/drupal-3504477/core/lib/Drupal/Core/Security/DoTrustedCallbackTrait.php:107
/builds/issue/drupal-3504477/core/lib/Drupal/Core/Render/Renderer.php:830
/builds/issue/drupal-3504477/core/lib/Drupal/Core/Render/Renderer.php:387
/builds/issue/drupal-3504477/core/lib/Drupal/Core/Render/Renderer.php:459
/builds/issue/drupal-3504477/core/lib/Drupal/Core/Render/Renderer.php:203
/builds/issue/drupal-3504477/core/tests/Drupal/KernelTests/Components/ComponentKernelTestBase.php:95
/builds/issue/drupal-3504477/core/lib/Drupal/Core/Render/Renderer.php:593
/builds/issue/drupal-3504477/core/tests/Drupal/KernelTests/Components/ComponentKernelTestBase.php:95
/builds/issue/drupal-3504477/core/tests/Drupal/KernelTests/Components/ComponentRenderTest.php:313
/builds/issue/drupal-3504477/core/tests/Drupal/KernelTests/Components/ComponentRenderTest.php:45
ERRORS!

Shows the test coverage

Believe this would solve a problem I see on my patterns theme were I have to pass #markup => content, in the twig file.

Change makes sense.

🇺🇸United States smustgrave

Can't re-run the test but failure appears to be in nightwatch /builds/project/drupal/core/modules/toolbar/tests/src/Nightwatch/Tests/toolbarTest.js:50

So pretty sure it's random.

Left a comment on the MR.

🇺🇸United States smustgrave

You’re completely good! And doesn’t bother anyone

If you click into the pipeline you’ll see a functional test for big auth is failing so the change may need to be tweaked

The functional JavaScript failure is most likely random and just needs to be re ran

🇺🇸United States smustgrave

Haven’t reviewed but issue summary appears incomplete

🇺🇸United States smustgrave

No worries @dcam happens all the time.

Am moving back to NW as there should be an MR and test

🇺🇸United States smustgrave

Big auth appears to be failing.
New parameters should be typehinted also

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

Seems like a simple change, IS is clear

1) Drupal\Tests\node\Functional\NodeEditFormTest::testNodeAuthorDisplayName
Behat\Mink\Exception\ExpectationException: The string "<em>3</em>" was not found anywhere in the HTML response of the current page.
/builds/issue/drupal-3183509/vendor/behat/mink/src/WebAssert.php:888
/builds/issue/drupal-3183509/vendor/behat/mink/src/WebAssert.php:363
/builds/issue/drupal-3183509/core/tests/Drupal/Tests/WebAssert.php:559
/builds/issue/drupal-3183509/core/modules/node/tests/src/Functional/NodeEditFormTest.php:320
FAILURES!
Tests: 4, Assertions: 105, Failures: 1.

Shows the test coverage

See no issue.

🇺🇸United States smustgrave

Added to the top of my list for tomorrow!

🇺🇸United States smustgrave

Ok in that case should this land now? Know there was a comment in the other about combining.

Applied the MR though and text-align right still appears to be working as before so marking RTBC if this should still land before.

🇺🇸United States smustgrave

Thanks for reporting.

Something that will be needed is a test case showing the issue.

🇺🇸United States smustgrave

Right but maybe we should.

Just to announce the new validation type that contrib modules can now use.

🇺🇸United States smustgrave

Some of the responses were from over a year ago but vs trying to get everyone to confirm again lets go with this. Verified all the names in the summary appear to have responded here.

🇺🇸United States smustgrave

This should probably have a CR right?

🇺🇸United States smustgrave

Fixed and added test coverage

🇺🇸United States smustgrave

Actually been a few years without movement wonder still needed feature?

🇺🇸United States smustgrave

Less steps the better I think as this could potentially break something, edge cases

🇺🇸United States smustgrave

Can we add post_update test coverage

🇺🇸United States smustgrave

Thanks believe feedback has been addressed and solution seems to address the issue.

🇺🇸United States smustgrave

Compromise lets add a small comment as to why are keeping

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

Believe to still be valid

🇺🇸United States smustgrave

Should have a test case showing the issue.

Also issue summary is incomplete.

🇺🇸United States smustgrave

Since it's been 2 years going to close out as things have changed.

But to answer, probably would use the settings to make sure sitemap and this line up. As far as automation there are a few sitemap modules out there so probably be hard to implement a one size fits all.

🇺🇸United States smustgrave

Odd request, could you add my company name to the module page with "Sponsoring co-maintainer + maintenance - smustgrave" helps justify paying for contribution time.

🇺🇸United States smustgrave

This appears to be opened against D7 of drupal which has seen hit EOL. If this needs to be reopened we can.

🇺🇸United States smustgrave

This appears to be opened against D7 of drupal which has seen hit EOL. If this needs to be reopened we can.

🇺🇸United States smustgrave

Doing the same over in https://www.drupal.org/project/cacheexclude/issues/3516075#comment-16047107 📌 Start a new branch + update gitlab Active to also expand gitlab

🇺🇸United States smustgrave

Going to mark this one as fixed and updating the gitlab pipeline file to run against different version of core.

🇺🇸United States smustgrave

Seemed to be answered in #2

🇺🇸United States smustgrave

So this issue appears to have been opened against D7, which has since hit EOL. Going to close out since it's been untouched in 12 years. But if we need to reopen we can.

🇺🇸United States smustgrave

Test failure seems relevant to the change.

🇺🇸United States smustgrave

So this issue appears to have been opened against D7, which has since hit EOL. Going to close out since it's been untouched in 12 years. But if we need to reopen we can.

🇺🇸United States smustgrave

So this issue appears to have been opened against D7, which has since hit EOL. Going to close out since it's been untouched in 12 years. But if we need to reopen we can.

🇺🇸United States smustgrave

Since there hasn't been a follow up going to close out. If still a valid request for D11 we can always reopen

🇺🇸United States smustgrave

Going to close this one out since there hasn't been a follow up and from what I can tell the installer has been working with php8

🇺🇸United States smustgrave

So I actually don't see any examples either, so wonder if this should be closed out or scope expanded? But yea that would be an undertaking and not sure needed? Could be wrong

Production build 0.71.5 2024