Skip to content
GitLab
Explore
Sign in
Primary navigation
Search or go to…
Project
eca-3482670
Manage
Activity
Members
Labels
Plan
Custom issue tracker
Code
Merge requests
0
Repository
Branches
Commits
Tags
Repository graph
Compare revisions
Locked files
Build
Pipelines
Jobs
Pipeline schedules
Artifacts
Deploy
Model registry
Analyze
Value stream analytics
Contributor analytics
CI/CD analytics
Repository analytics
Code review analytics
Insights
Model experiments
Help
Help
Support
GitLab documentation
Compare GitLab plans
Community forum
Contribute to GitLab
Provide feedback
Terms and privacy
Keyboard shortcuts
?
Snippets
Groups
Projects
Show more breadcrumbs
Issue forks
eca-3482670
Commits
4b5e4f4b
Commit
4b5e4f4b
authored
2 years ago
by
Jürgen Haas
Browse files
Options
Downloads
Patches
Plain Diff
Issue
#3306135
: Validate plugin has redundant code for "replace_tokens" field
parent
638f9f55
No related branches found
Branches containing commit
Tags
8.x-1.0
Tags containing commit
No related merge requests found
Changes
1
Hide whitespace changes
Inline
Side-by-side
Showing
1 changed file
src/Entity/Eca.php
+0
-10
0 additions, 10 deletions
src/Entity/Eca.php
with
0 additions
and
10 deletions
src/Entity/Eca.php
+
0
−
10
View file @
4b5e4f4b
...
...
@@ -451,16 +451,6 @@ class Eca extends ConfigEntityBase implements EntityWithPluginCollectionInterfac
$fields
[
$key
]
=
mb_strtolower
(
$fields
[
$key
])
===
'yes'
;
}
}
// When "replace_tokens" exists, also convert that one back to boolean.
// @see \Drupal\eca\Service\Actions::getConfigurationForm()
if
(
isset
(
$fields
[
'replace_tokens'
]))
{
if
(
!
empty
(
$fields
[
'replace_tokens'
])
&&
is_string
(
$fields
[
'replace_tokens'
]))
{
$fields
[
'replace_tokens'
]
=
mb_strtolower
(
$fields
[
'replace_tokens'
])
===
'yes'
;
}
else
{
$fields
[
'replace_tokens'
]
=
(
bool
)
$fields
[
'replace_tokens'
];
}
}
}
// Build form.
$form_state
=
new
FormState
();
...
...
This diff is collapsed.
Click to expand it.
Preview
0%
Loading
Try again
or
attach a new file
.
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Save comment
Cancel
Please
register
or
sign in
to comment