Account created on 7 October 2022, about 2 years ago
#

Merge Requests

More

Recent comments

🇨🇷Costa Rica yuvania

I have the same problem as @sheetal.pathak and I couldn't reproduce the error. In my case I used a 10.3.5 environment.

🇨🇷Costa Rica yuvania

I couldn't recreate the issue on my end. Could you share a screenshot of your current configuration to see if there's any difference?

🇨🇷Costa Rica yuvania

Thanks for the merge request. I tested it and it works perfectly. Ready to move forward!

🇨🇷Costa Rica yuvania

I tried it on drupal 10.3 and it worked fine for me

🇨🇷Costa Rica yuvania

I confirm, I used it yesterday and it worked perfectly.

🇨🇷Costa Rica yuvania

Reviewed and tested the patch. Everything works as expected.

🇨🇷Costa Rica yuvania

yuvania made their first commit to this issue’s fork.

🇨🇷Costa Rica yuvania

Agree with chandu7929, the changes look good and work well.

🇨🇷Costa Rica yuvania

yuvania made their first commit to this issue’s fork.

🇨🇷Costa Rica yuvania

You could perhaps try using an older version of Drush, such as 12.x, which is known to be stable with Drupal 10.3, to see if that resolves the issue

🇨🇷Costa Rica yuvania

I've tested the latest changes in my environment and everything seems to be working perfectly. Thanks everyone for the great work!

🇨🇷Costa Rica yuvania

Maybe you could try breaking down the target_id further in the errorElement function to see if something is being missed in the validation. I'm referring to checking or adjusting the structure of $element before it reaches the check, in case there's any data not being handled correctly

🇨🇷Costa Rica yuvania

Did you notice if this issue started after an update? You might also want to check the logs and JavaScript console for specific errors, and maybe try disabling the cache temporarily to see if that helps isolate the cause

🇨🇷Costa Rica yuvania

Good eye on this, I think that fix will help the field_tags look better in the teaser. Thanks for sharing!

🇨🇷Costa Rica yuvania

To fix the "headers already sent" issue, I added a check in the BigPipe.php file to make sure the headers hadn’t already been sent before printing the content:

if (!headers_sent()) {
    if ($chunk instanceof HtmlResponse) {
        print $chunk->getContent();
    } else {
        print $chunk;
    }
    flush();
} else {
    // You can handle the situation here if the headers have already been sent
}

I also tried wrapping the code in an output buffer to prevent anything from being sent too early, which helped avoid the error:

ob_start();

// Here’s the code that sends the headers or starts a session
if ($chunk instanceof HtmlResponse) {
    print $chunk->getContent();
} else {
    print $chunk;
}
flush();

ob_end_flush();
🇨🇷Costa Rica yuvania

Hello alexander tallqvist , I tested the patch, and it worked for me.

🇨🇷Costa Rica yuvania

Hi @aaron.ferris ,

  1. I tested the change https://git.drupalcode.org/issue/views_ical-3465114/-/commit/b718fd9d626... where the property vTimezone was added to the style, and it works perfectly. I had considered the same solution, and here's why it makes sense:
  2. Avoiding Deprecated Dynamic Properties: By defining the property explicitly in the class, we prevent PHP 8.3's deprecation warnings about dynamic property creation. This keeps our code future-proof and aligns with best practices.
  3. Logical Association: vTimezone is intrinsically tied to the style's configuration and state. Placing it within the style ensures that all relevant data and behavior are encapsulated within the appropriate object.
  4. Maintaining Functionality: This change preserves the existing functionality and ensures that time zone transitions are correctly managed and applied within the iCal generation.

Thanks for the merge request! It should help maintain compatibility with newer PHP versions while keeping the module's functionality intact.

🇨🇷Costa Rica yuvania

Hi @droodpal,

I've reviewed the pages and compared the styles, but I wasn't able to reproduce the error. The styles for the problematic page and the one that works correctly seem to be the same.

Here are a few suggestions you might want to consider:

  1. Browser Caching: Ensure your browser cache is cleared. Sometimes old styles can be cached and affect how the page is displayed.
  2. CSS Specificity: Double-check if there are any specific styles being overridden by other CSS rules. Use browser developer tools to inspect the styles applied to the elements.
  3. Custom CSS or JS: Verify if there is any custom CSS or JavaScript that might be interfering with the layout on the problematic page.
  4. Responsive Design: Check if there are any media queries or responsive design rules that might be affecting the display on certain screen sizes or devices.
  5. Modules and Themes: Ensure that all modules and themes are up to date. Sometimes issues arise from outdated components.
  6. Error Logs: Look at the browser console for any errors or warnings that might give a clue about what's going wrong.
  7. Class Conflicts: Make sure there are no conflicting class names that might be causing unintended styles to be applied.

Additionally, I'll check if anyone has commented elsewhere on the internet about a similar issue to see if there's a known solution or workaround.

Hope this helps!

🇨🇷Costa Rica yuvania

Hi @Pemson18,

I've reviewed the discussion, and I'd like to offer a suggestion that might help clarify the process of block instantiation and placement using code in Drupal 10.

From my experience, it is possible to create and place blocks programmatically without using the UI configuration. Here's a basic example of how you might achieve this in an update hook:

use Drupal\block\Entity\Block;

/**
 * Implements hook_update_N().
 */
function mymodule_update_10001() {
  // Create a new custom block.
  $block = Block::create([
    'id' => 'my_custom_block',
    'theme' => 'bartik',
    'plugin' => 'my_custom_block_plugin',
    'settings' => [],
    'visibility' => [],
    'region' => 'sidebar_first',
    'weight' => 0,
  ]);
  $block->save();

  // Ensure the block is placed in the desired region.
  \Drupal::service('theme_handler')->rebuildThemeData();
  \Drupal::service('theme_handler')->refreshInfo();
  $theme = \Drupal::theme()->getActiveTheme();
  $theme_name = $theme->getName();

  // Clear cache to apply changes.
  \Drupal::service('cache.render')->invalidateAll();
}

This code snippet creates a custom block and places it in the 'sidebar_first' region of the 'bartik' theme. Note that you need to adjust 'my_custom_block_plugin' to your actual block plugin ID.

Make sure to clear the cache after running the update hook to see the changes. This approach should bypass the need for UI configuration and ensure that your block is placed directly via code.

I hope this helps clarify the problem! Let me know if you need further clarification or assistance.

🇨🇷Costa Rica yuvania

Hi @eaajithe,

What an interesting issue! Based on your description, here are a few steps that might help resolve it:

  1. Field Configuration: Ensure the email field is set as required in the user entity field settings. Sometimes configurations might change during upgrades.
  2. Module Interference: Check if modules like "Password Policy" or others affecting user profiles are causing the issue. Temporarily disable them to test.
  3. Permissions: Verify user roles have the necessary permissions to edit and view the email field.
  4. Update Hooks: Confirm all update hooks have run correctly after upgrading from Drupal 9 to Drupal 10.
  5. Form Display: Check the form display settings to ensure the email field is included.

These steps should help identify the root cause.

Hope this helps!

🇨🇷Costa Rica yuvania

I've tested the provided patch and observed a significant improvement in the load times for the admin/config page. This change greatly enhances the usability of the configuration page in larger projects.

I agree with the suggestion to run these checks less frequently, such as through a cron job or lazy loading, to ensure we still catch potential issues without affecting performance. Implementing this would provide a balanced approach to maintain site health without compromising speed.

🇨🇷Costa Rica yuvania

The patch improves the infinite-scroll.js file and the views_infinite_scroll.module to ensure that infinite scrolling works consistently. Changes include:

Adding the once parameter to the JavaScript function to ensure certain behaviors are applied only once per element.
Adjusting the module to attach the views-infinite-scroll library only when AJAX is enabled on a view.

🇨🇷Costa Rica yuvania

yuvania made their first commit to this issue’s fork.

🇨🇷Costa Rica yuvania

I fixed access check errors for D10 compatibility

🇨🇷Costa Rica yuvania

Patch to resolve compatibilities with drupal 10

🇨🇷Costa Rica yuvania

Patch to resolve Fix phpunit 10 obsolescence

🇨🇷Costa Rica yuvania

Patch to resolve Fix phpunit 10 obsolescence

🇨🇷Costa Rica yuvania

https://www.drupal.org/project/ultimate_cron/issues/3394314#comment-1527... 🐛 Deprecated method implementsHook() Active In this issue, I added a patch that fixes it!

🇨🇷Costa Rica yuvania

This is a better version than the previous patch. The previous one improved the code but did not solve the problem of the deprecated method: implementsHook()

🇨🇷Costa Rica yuvania

Using this patch I solved broken code problems, I leave it here in case anyone needs it

🇨🇷Costa Rica yuvania

Patch to fix compatibility issues with Drupal 10

🇨🇷Costa Rica yuvania

yuvania made their first commit to this issue’s fork.

🇨🇷Costa Rica yuvania

I checked it, it works fine for me.

🇨🇷Costa Rica yuvania

I checked it, it works fine for me.

🇨🇷Costa Rica yuvania

You have a point, I checked it, it worked fine for me.

🇨🇷Costa Rica yuvania

You have a point, I checked it, it worked fine for me.

🇨🇷Costa Rica yuvania

Hello, I tried the patch and it works correctly.

Production build 0.71.5 2024