haritha c → made their first commit to this issue’s fork.
Hi @jorgik,
I've fixed the code violation issues. Kindly review and let me know if any further changes are required.
haritha c → changed the visibility of the branch 3515395-coding-voilation to hidden.
haritha c → made their first commit to this issue’s fork.
haritha c → changed the visibility of the branch 3515395-fix-coding-standards to active.
haritha c → changed the visibility of the branch 3515395-fix-coding-standards to hidden.
haritha c → made their first commit to this issue’s fork.
Hi @prashant.c,
Thanks for your response.
I've resolved all the pipeline and PHPCS issues in this merge request itself:
🔗 https://git.drupalcode.org/issue/ai_search_block-3519312/-/tree/3519312-...
Kindly review and let me know if anything else is needed.
I have updated the code to make the "Source database" field required in the block configuration form by adding:
'#required' => TRUE
This ensures that the search block cannot be saved without selecting a source, which helps prevent misconfiguration.
However, the merge request pipeline initially failed due to PHPStan static analysis and/or PHPCS code style issues.
If anyone could guide me on how to resolve this pipeline issue, I would greatly appreciate it.
Thank you!
haritha c → changed the visibility of the branch 3519312-source-database to hidden.
haritha c → changed the visibility of the branch 3519312-source-database-should to active.
haritha c → changed the visibility of the branch 3519312-source-database-should to hidden.
haritha c → changed the visibility of the branch 3519312-source-database-should to hidden.
haritha c → made their first commit to this issue’s fork.
Thank you for your comment. I've removed the .patch file from the updated code repository.
I have fixed the tab memory issue by converting $cookieValue to an integer using parseInt($cookieValue, 10).
The fix is committed in the branch 3519116-tab-memory-feature.
Patch is also uploaded.
https://git.drupalcode.org/issue/quicktabs-3519116/-/tree/3519116-tab-me...
Please review.
haritha c → made their first commit to this issue’s fork.
Updated login_switch.info.yml by replacing configure: login_switch_admin with configure: login_switch.settings to match the route defined in login_switch.links.menu.yml. The module now installs and functions as expected.
Patch attached and also committed to the Git repo (9710debb) with the message: "Fixed the error on install."
haritha c → made their first commit to this issue’s fork.
Its verified , filename was updated and working as expected
haritha c → changed the visibility of the branch 3518702-imagelightbox-file-rename to hidden.
haritha c → made their first commit to this issue’s fork.
I’ve fixed this issue by allowing the default "Save" button text to be overridden.
The fix is available in the branch [3514815-allow-override-of](https://git.drupalcode.org/issue/save_edit-3514815/-/tree/3514815-allow-...). Tested on Drupal 10. Please review!
haritha c → changed the visibility of the branch 3514918-add-gitci to hidden.
haritha c → changed the visibility of the branch 3514918-add-gitci to hidden.
haritha c → changed the visibility of the branch 3514918-add-gitci to hidden.
haritha c → made their first commit to this issue’s fork.
Hi,I tested this on Drupal 11 with Save & Edit 1.8, and it works as expected. Here’s how you can enable and use it properly:
Steps to Enable & Configure:
1) Install & Enable the Module:
composer require 'drupal/save_edit:^1.8'
drush en save_edit -y
drush cache:rebuild
2)Go to Configuration Page:
Navigate to Configuration → Content Authoring → Save & Edit
(URL: /admin/config/content/save_and_edit)
Here, you can rename the Save & Edit button to your preferred text.
Important: Check if you have enabled Save & Edit for the specific content type in the settings.
3)Using the Buttons in Content Edit:
Go to Content → Add or Edit any node.
After clicking Preview, you will see:
Save & Edit
Save
Preview
Back to Content Edit (when returning from preview mode).
Clicking Save & Edit saves the content and keeps you on the edit page.
If You Don't See the Buttons:
-----------------------------------------
Ensure the module is enabled.
Verify that Save & Edit is enabled for your content type in the module settings.
Clear the cache (drush cache:rebuild).
haritha c → changed the visibility of the branch 3511454-fixes_field_ui_entity_issues to active.
haritha c → changed the visibility of the branch 3511454-fixes_field_ui_entity_issues to hidden.
I have fixed the PHPCS and unit test issues for smart_trim-3512933. Please review and let me know if any further changes are needed.
I have updated the issue in this module ,pls review it.
haritha c → made their first commit to this issue’s fork.
haritha c → created an issue.
I have worked on resolving the reported issues. Some of the PHPUnit test issues are fixed, but a few still remain. However, the functionality is working fine. Locally, my code does not show any errors, but I am still facing issues in the pipeline. Could you provide any suggestions on how to debug or resolve this? Thanks!
Fixed issue #3512933—'More' link now only shows if visible text exceeds the trim limit. For short content (e.g., 20 chars) with a 50-char limit, it no longer displays . Patch (version 5) from branch 3512933-smart_trim_fixes attached.
haritha c → made their first commit to this issue’s fork.
The issue has been fixed, and I have updated the patch in Git (21df20b0) with the border color adjustments for the tiles.
Please review and update the status accordingly.
haritha c → made their first commit to this issue’s fork.
I tested this issue using Drupal 10.4 with UI Patterns 2.0 installed via:
composer require 'drupal/ui_patterns:^2.0'
Steps I followed:
Installed UI Patterns and enabled necessary submodules.
Created a View and added UI Patterns as a row format.
Successfully added and removed fields from the View.
Checked Layout Builder integration, and I was able to add and remove fields from UI Patterns slots.
Cleared cache using drush cache:rebuild, and everything worked fine.
Conclusion:
I couldn't reproduce the issue. UI Patterns fields are appearing correctly in Views and Layout Builder.
Can you check if you're using the latest version and clear caches (drush cr) to see if the issue persists?
I tried to reproduce this issue on Drupal 10.4 with Styled Google Map 8.x-2.6.
Installed via composer require drupal/styled_google_map:^2.6
Created a block with a Geofield (latitude/longitude)
Configured Google Maps API key under Admin > Configuration > Styled Google Map
Map displayed correctly without errors
No error was found in logs, and the configuration page loaded correctly.
Can you share more details like:
The Drupal version you are using
Exact steps to reproduce the issue
The error message from logs (if any)
okey ,Thanks for the reply
I have tested the fix in Drupal 11.1.3 and confirmed that everything works as expected. @rcodina, could you please check and add my contribution under ‘Reviewed & tested by’?
Thank you!
Tested & Works as Expected
I have tested commit b58bf7ee2519999f42d01fc16c6b978573b6143b from the branch faqeh-2.0.0-beta3-patch-c221.
The success message now displays correctly after creating a subgroup.The redirect works fine without any issues.
I have tested the changes from the issue fork 3496973-fix-issues-reported by cloning the repository and updating the toolbar_menu_clean module in my Drupal setup.
After running the PHPCS command:
phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml toolbar_menu_clean/
No PHPCS errors or warnings were found.
The issue appears to be resolved, and the code follows Drupal coding standards.
Status: Works as expected.
Tested the fix in Drupal 11.1.3
The core_version_requirement update works correctly.
Composer allows installation and upgrades beyond Drupal 11.
No issues found.
Ready to be merged.
haritha c → made their first commit to this issue’s fork.
haritha c → changed the visibility of the branch 3508992-clear-button-does to active.
haritha c → changed the visibility of the branch 3508992-clear-button-does to active.
haritha c → changed the visibility of the branch 3508992-clear-button-does to hidden.