-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Contribution] DARWIN-1045: Provide link to user profile and/or subscriptions page #34
base: 1.x
Are you sure you want to change the base?
Conversation
if ( | ||
$type == 'oe_subscriptions_anonymous' && | ||
isset($tokens['subscriptions_page']) && | ||
!empty($data['message']) | ||
) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like this formatting better:
if ( | |
$type == 'oe_subscriptions_anonymous' && | |
isset($tokens['subscriptions_page']) && | |
!empty($data['message']) | |
) { | |
if ($type === 'oe_subscriptions_anonymous' | |
&& isset($tokens['subscriptions_page']) | |
&& !empty($data['message']) | |
) { |
Actually, Drupal core is inconsistent about this.
And in /vendor/ you also find different formats.
See
https://www.drupal.org/project/coding_standards/issues/3392321
https://www.drupal.org/project/coding_standards/issues/1539712
Look for "&&" to find relevant comments.
Ultimately you can decide this with the OEL team.
@@ -50,7 +63,8 @@ public function testLinkTemplate(): void { | |||
'', | |||
[ | |||
'href' => $front_url->toString(), | |||
] | |||
], | |||
$template_name |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Multi line argument list should have comma after the last item.
I thought this would be in automatic code style checks but seems it is not?
7f5633a
to
74e66fc
Compare
No description provided.