This issue actually changes quite a bit more:

  • Moves the logic of determining the current group from domain to a dedicated domain group info service that is reused in different places and can be used by third party code,
  • adds additional automated tests (thanks to chx),
  • adds a default Views ardument for the current domain group,
  • makes the code compliant with Drupal, DrupalPractice and strict types standards

Issue fork group_domain-3296013

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Graber created an issue. See original summary.

Graber opened merge request !3

Graber credited Charlie ChX Negyesi.

Graber’s picture

Graber CreditAttribution: Graber as a volunteer commented
Title: Move group / domain info logic to a service » Move group / domain info logic to a service, add Views default argument
Issue summary: View changes

Graber credited catch.

Graber’s picture

Graber CreditAttribution: Graber at Tag1 Consulting for American Federation of Teachers commented
Assigned: Graber » Unassigned
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.