Skip to content
Snippets Groups Projects

Issue #3448265 by n.ghunaim: Fix an Error when using Layout Builder permissions with the Section Library module

Issue #3448265 by n.ghunaim: Fix an Error when using Layout Builder permissions with the Section Library module
issue/section_library-3448265:3448265-error-when-using-permissions into 1.0.x
1 open thread
Closed
Noor Ghunaimrequested to merge
issue/section_library-3448265:3448265-error-when-using-permissions into 1.0.x
1 open thread

Checkpoints

  • File an issue
  • Addition/Change/Update/Fix
  • Merge request
  • Testing to ensure no regression
  • Automated unit testing coverage
  • Automated functional testing coverage
  • UX/UI designer responsibilities
  • Readability
  • Accessibility
  • Performance
  • Security
  • Documentation
  • Code review by maintainers
  • Full testing and approval
  • Credit contributors
  • Review with the product owner

Closes #3448265

Edited by Rajab Natshah

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Rajab Natshah changed title from Update SectionLibraryRender.php to Issue #3448265 by n.ghunaim: Fix an Error when using Layout Builder permissions with the Section Library module

    changed title from Update SectionLibraryRender.php to Issue #3448265 by n.ghunaim: Fix an Error when using Layout Builder permissions with the Section Library module

  • Rajab Natshah approved this merge request

    approved this merge request

  • Rajab Natshah marked the checklist item Code review by maintainers as completed

    marked the checklist item Code review by maintainers as completed

  • Rajab Natshah marked the checklist item Performance as completed

    marked the checklist item Performance as completed

  • Rajab Natshah marked the checklist item Security as completed

    marked the checklist item Security as completed

  • Rajab Natshah
    Rajab Natshah @RajabNatshah started a thread on the diff
src/SectionLibraryRender.php
114 114 }
115 115
116 $current_route_match = \Drupal::service('current_route_match');
117 $route_name = $current_route_match->getRouteName();
118 $params = $current_route_match->getParameters()->all();
119
120 // Add save template to library link to the template.
116 121 $add_template_to_library_url = Url::fromRoute(
117 122 'section_library.add_template_to_library',
118 123 [
119 'section_storage_type' => $params['section_storage_type'],
120 'section_storage' => $params['section_storage'],
121 'delta' => $params['delta'],
124 'section_storage_type' => $params['section_storage']->getStorageType(),
125 'section_storage' => $params['section_storage']->getStorageId(),
126 'delta' => count($params['section_storage']->getSections()),
  • Jess Straatmann
    Jess Straatmann @jastraat
    Maintainer

    Closing merge request as outdated -

  • Jess Straatmann closed

    closed

    • Please register or sign in to reply