@manibharathi ezhimalai ravi

Hi,
Since not able to create the branch for the latest version Drupal 11.x.

So Attaching the rerolled patch file for the core version 11.x.

Hi,
Already the issue is noticed and we have patch in the ticket https://www.drupal.org/project/drupal/issues/2358537 ✨ Do not require a 'title' field Needs work . Rerolled the patch also for the latest version in that ticket.

So instead duplicate we can close this ticket.

Hi,
Since not able to create the branch for the latest version Drupal 11.x.

So Attaching the rerolled patch file for the core version 11.x.

Hi,
I am not able to reproduce the issue in the Drupal 10.4.0-dev Version.

Steps Followed:
1. Disabled the standalone in the media settings /admin/config/media/media-settings
2. Editing the any media item. Primary options tabs are visible for the user.
3. Enabled the standalone in the media settings
4. Able to see the primary tabs in the options.

As per the Implementation of the "Standalone media setting" working as per the settings. Media URL view option is visible when the settings is enabled. If that setting is disabled the except view option will display the other primary tabs options.

Before Enable the Standalone:

After Enable the Standalone:

Let me know incase of anything I missed in this steps.

Hi,
Able to reproduce the issue in the Drupal11.x. After creating the view with specific html element data is not coming for the iframe tag.

MR !4595 has been verified in the Drupal 11.x.

Before Patch:

After Patch:

Hi,
Created the MR for the changes and updated the testcase for the new field adding. Please letme know incase of anything needs to be update for the changes.

This how we display the after selecting the promoted option in the menu settings.

manibharathi ezhimalai ravi → changed the visibility of the branch 2518960- to hidden.

manibharathi ezhimalai ravi → made their first commit to this issue’s fork.

breidert → credited manibharathi ezhimalai ravi → .

Hi,
MR !8815 has been verified successfully in the Drupal 11.x version. Since all the testcases are passed. Nothing to be update for the functionality.
Let me know incase of anything need to be update.

Before Patch:

After Patch:

Hi,
MR has been created for the changes. Since the issue is reproducible in the 2.0.x. Created the MR with the latest branch.

manibharathi ezhimalai ravi → made their first commit to this issue’s fork.

Hi,
Since this issue is already reported and functionality is also worked completed in the previous issue Add permission for showing only the secondary toolbar ✨ Add permission for showing only the secondary toolbar Needs work .

Please confirm this ticket is required since we already added the permission and checking permission for the user is already done.

Hi,
For fixing the conflicts created the separated branch from the issue fork. Changes are working as expected in the MR !50.

New permissions are added for the access the toolbar.

manibharathi ezhimalai ravi → made their first commit to this issue’s fork.

Hi,
MR !9668 has been verified successfully. Attributes are removed from the date field.

Before Patch:

After Patch:

Hi,
Changes has been verified in the issue fork branch. working has expected to the branch.
Before Branch:

After: Branch:

But the patch 9341.patch is failing in the Drupal 11.x branch. Please find the screenshot for the more understanding.

Hi,
Issue is reproducible in the 4.0.
MR !513 has been created for the 4.0. Changes updated with the sidebar section is exist with the condition.

Pending Task:

• Team feedback
• UI fixes

manibharathi ezhimalai ravi → made their first commit to this issue’s fork.

Hi,
I am able to reproduce the issue. Done manual testing for the Drupa Version 11.x.

Steps followed for the Update:

1. Install docker image with the php 8.3
2. Go inside the docker container
3. mkdir drupal11_l10n_test && cd drupal11_l10n_test
4. composer create-project drupal/recommended-project .
5. # Installing drupal/recommended-project (11.0.1)
6. composer require --dev drupal/core-dev
7. composer require drush/drush
8. composer require 'drupal/sqlite337:^1.0@alpha'
9. mkdir -p db
10. # triple '../' works in Drupal 11 (single '../' worked in Drupal 10) ./vendor/bin/drush site:install --yes --db-url="sqlite://../../../db/test.sqlite?module=sqlite337" --site-mail noreply@example.com

MR !9431 applied successfully. Changes looking good to me.

Before Patch:

After Patch:

Hi,
Updated the MR with _auth check condition instead user checking.

_auth checking condition is added based on the #23 🐛 Basic auth returns 403 when username & password supplied but not needed. Needs work and also feedback received in the thread.

Hi,
MR has been created !9584 for the changes. Testcases also passed in the pipeline. Let me know incase of any new changes needs to be add.

tonypaulbarker → credited manibharathi ezhimalai ravi → .

manibharathi ezhimalai ravi → made their first commit to this issue’s fork.

Hi
I tried the following methods to check whether the permission for the content moderation in editorial user.

1. Created a new service for checking the permission with the user that the access or moderation field is disabled or not in the condition.
2. In the validator check that the user has the permission and the content moderation state is disabled for the user or not.
3. In the validator itself check that user workflow moderation content state for permission is allowed or not.

This is the Code is already checking the permission. Even the conditon for checking permission also working properly but addviolation constraint message is not coming.

if (!$this->stateTransitionValidation->isTransitionValid($workflow, $original_state, $new_state, $this->currentUser, $entity)) {
        $this->context->addViolation($constraint->invalidTransitionAccess, [
          '%original_state' => $original_state->label(),
          '%new_state' => $new_state->label(),
        ]);
      }

Please let me know incase if I miss anything or Can suggest any new methods to fix the issue.

Hi,
Created the MR !9539 for the changes. Mentioned in the #34 🐛 Drupal\media\Entity\Media::prepareSave should convert URL object metadata to string before saving Needs review already verified that field is instance of the URL or not with the condition.
let me know incase of I missing with adding condition.

Hi,
If we update the hidden or disable the moderation state field in the content form. If user want to change the state from one state to another state is not possible for the editor.

Hi,
Created the Merge request !9462 for the Zebra
counting of nodes.

As per mentioned in the comment #3 💬 Zebra counting of nodes not available Active ticket is in support request. We can use this merge request for adding the Zebra variable for the node page.

manibharathi ezhimalai ravi → made their first commit to this issue’s fork.

Updated the code changes for the _auth check for the current page route exist with the basic_auth or not.

but testcase failing in the basicauthtest. Attaching the screenshot of the unit testcase fails.

Hi,
Trying to add the condition whether the current request is coming with the options _auth and value should basic_auth. But getting the test failure after implementing the changes.

Hi,
By default Drupal will check the all the route values. Since we checking credentials from the request header all the URL's should be verified.

mstrelan, Let me know Incase of missing any functionality check for this implementation.

Hi,
I have created the MR !9389 for this feature.

Before Patch:

After Patch:

manibharathi ezhimalai ravi → made their first commit to this issue’s fork.

Hi,
I am not able reproduce the issue in the Drupal 10.x and also 11.x.

Steps Followed:

1. Install Drupal with the different language English-british or other language in the installation.
2. Uninstall the language module
3. Tried to update the any user
4. Able to save the user information in the edit page.

kindly let me know incase if i missing any steps for the reproduce.

Hi,
I have created the MR !9368 for the Branch 11.x.

manibharathi ezhimalai ravi → changed the visibility of the branch 3468755- to hidden.

Hi,
I have created the MR !9351 for this feature.

manibharathi ezhimalai ravi → made their first commit to this issue’s fork.

Hi,
Issue able to reproduce in the Drupal10.3.x.
MR !8585 has been fixing the issue for Drupal 10.3.x.

Issue not able to reproduce in the Drupal 11.x. Please find the evidence from the Drupal11.x.

Steps to Reproduce the issue:

1. Install Drupal 10.3.x
2. Enable the Settings Tray and Layout builder module
3. Enable the layout builder for any content type
4. Add the two different inline blocks in the layout settings
5. Check in the console and observe the issue.

Since this issue is already reviewed we can move to the RTBC.

manibharathi ezhimalai ravi → changed the visibility of the branch 3458067-contextual-links-disappear-11.x to hidden.

Hi,
I've reviewed the MR !9183 on Drupal 11.x.

Layout Discard changes opened in the dialog box.

Hi,
I've reviewed the MR !9233 on Drupal 11.x.

Changes Looks good to me. [view:title] token values are render properly.

Hi,
I've reviewed the MR !5328 on Drupal 11.x.

Test Results:
Non link text is appearing without any link styles in the content.

Hi,
I am not able to reproduce the issue in the Drupal 11.x-dev version.

Steps Followed as per the commands #7. Both safari and Firefox browser are Fullscreen working properly for the oembed videos.

Hi,
I've reviewed the MR !9172 on Drupal 11.x.dev

Testing Steps

1. Enabled Interface Translation module
2. Went to admin/config/regional/translate
3. Verify the note is updated.

Testing Results
Translation page help note is updated based on the comment #66.

Hi,
I've reviewed the MR !9173 on Drupal 10.3.

Testing Steps:

1. Create a view of content, it could be Page or Block
2. Set the pager to full pager and show two items
3. Look at the view output in Olivero with full pager and then with mini

Test Results

Content are looking properly displaying inline in the mini pager and Full pager settings.