Remove 'edit only' blocking statements. (CFM-218)

COmanage · Sep 24, 2025 · f32493b · f32493b
1 parent 093219c
commit f32493b
Show file tree

Hide file tree

Showing 18 changed files with 0 additions and 90 deletions.
diff --git a/app/availableplugins/ApiConnector/templates/ApiSources/fields.inc b/app/availableplugins/ApiConnector/templates/ApiSources/fields.inc
@@ -25,11 +25,6 @@
  * @license       Apache License, Version 2.0 (http://www.apache.org/licenses/LICENSE-2.0)
  */
 
-// This view does currently not support read-only
-if($vv_action !== 'edit') {
-  return;
-}
-
 // XXX https://todos.internet2.edu/browse/CFM-465 send type - this should be 'information' (not a warning)
 $banners = [
   __d('information', 'plugin.config.none')

diff --git a/app/plugins/CoreApi/templates/MatchCallbacks/fields.inc b/app/plugins/CoreApi/templates/MatchCallbacks/fields.inc
@@ -25,11 +25,6 @@
  * @license       Apache License, Version 2.0 (http://www.apache.org/licenses/LICENSE-2.0)
  */
 
-// This view does currently not support read-only, and add is not used
-if($vv_action !== 'edit') {
-  return;
-}
-
 if(!empty($vv_api_endpoint)) {
   // XXX https://todos.internet2.edu/browse/CFM-465 send type - this should be 'information' (not a warning)
   $banners = [

diff --git a/app/plugins/CoreAssigner/templates/FormatAssigners/fields.inc b/app/plugins/CoreAssigner/templates/FormatAssigners/fields.inc
@@ -25,11 +25,6 @@
  * @license       Apache License, Version 2.0 (http://www.apache.org/licenses/LICENSE-2.0)
  */
 
-// This view does currently not support read-only, and add is not used
-if($vv_action !== 'edit') {
-  return;
-}
-
 $fields = [
   'format',
   'collision_mode',

diff --git a/app/plugins/CoreAssigner/templates/SqlAssigners/fields.inc b/app/plugins/CoreAssigner/templates/SqlAssigners/fields.inc
@@ -25,11 +25,6 @@
  * @license       Apache License, Version 2.0 (http://www.apache.org/licenses/LICENSE-2.0)
  */
 
-// This view does currently not support read-only, and add is not used
-if($vv_action !== 'edit') {
-  return;
-}
-
 $fields = [
   'server_id',
   'source_table',

diff --git a/app/plugins/CoreEnroller/templates/ApprovalCollectors/fields.inc b/app/plugins/CoreEnroller/templates/ApprovalCollectors/fields.inc
@@ -25,11 +25,6 @@
  * @license       Apache License, Version 2.0 (http://www.apache.org/licenses/LICENSE-2.0)
  */
 
-// This view only supports edit
-if($vv_action !== 'edit') {
-  return;
-}
-
 $fields = [
   'require_comment',
   'denial_message_template_id',

diff --git a/app/plugins/CoreEnroller/templates/AttributeCollectors/fields.inc b/app/plugins/CoreEnroller/templates/AttributeCollectors/fields.inc
@@ -27,11 +27,6 @@
 
 declare(strict_types = 1);
 
-// This view only supports edit
-if($vv_action !== 'edit') {
-  return;
-}
-
 $fields = [
   'enable_person_find',
 ];

diff --git a/app/plugins/CoreEnroller/templates/BasicAttributeCollectors/fields.inc b/app/plugins/CoreEnroller/templates/BasicAttributeCollectors/fields.inc
@@ -25,11 +25,6 @@
  * @license       Apache License, Version 2.0 (http://www.apache.org/licenses/LICENSE-2.0)
  */
 
-// This view only supports edit
-if($vv_action !== 'edit') {
-  return;
-}  
-
 $fields = [
   'name_type_id',
   'email_address_type_id',

diff --git a/app/plugins/CoreEnroller/templates/EmailVerifiers/fields.inc b/app/plugins/CoreEnroller/templates/EmailVerifiers/fields.inc
@@ -25,11 +25,6 @@
  * @license       Apache License, Version 2.0 (http://www.apache.org/licenses/LICENSE-2.0)
  */
 
-// This view only supports edit
-if($vv_action !== 'edit') {
-  return;
-}
-
 // Used for placeholders
 $defaultValues = array_combine(
   array_values($defaults),

diff --git a/app/plugins/CoreEnroller/templates/IdentifierCollectors/fields.inc b/app/plugins/CoreEnroller/templates/IdentifierCollectors/fields.inc
@@ -25,11 +25,6 @@
  * @license       Apache License, Version 2.0 (http://www.apache.org/licenses/LICENSE-2.0)
  */
 
-// This view only supports edit
-if($vv_action !== 'edit') {
-  return;
-}
-
 $fields = [
   'type_id'
 ];
diff --git a/app/plugins/CoreEnroller/templates/InvitationAccepters/fields.inc b/app/plugins/CoreEnroller/templates/InvitationAccepters/fields.inc
@@ -25,11 +25,6 @@
  * @license       Apache License, Version 2.0 (http://www.apache.org/licenses/LICENSE-2.0)
  */
 
-// This view only supports edit
-if($vv_action !== 'edit') {
-  return;
-}
-
 $fields = [
   'invitation_validity' => [
     'default'   => 1440

diff --git a/app/plugins/CoreServer/templates/HttpServers/fields.inc b/app/plugins/CoreServer/templates/HttpServers/fields.inc
@@ -25,11 +25,6 @@
  * @license       Apache License, Version 2.0 (http://www.apache.org/licenses/LICENSE-2.0)
  */
 
-// This view does currently not support read-only, and add is not used
-if($vv_action !== 'edit') {
-  return;
-}  
-
 $fields = [
   'url',
   'username',

diff --git a/app/plugins/CoreServer/templates/Oauth2Servers/fields.inc b/app/plugins/CoreServer/templates/Oauth2Servers/fields.inc
@@ -25,11 +25,6 @@
  * @license       Apache License, Version 2.0 (http://www.apache.org/licenses/LICENSE-2.0)
  */
 
-// This view does currently not support read-only, and add is not used
-if($vv_action !== 'edit') {
-  return;
-}
-
 // Render the Redirect URI
 $fields = [
   'redirect_uri' => [

diff --git a/app/plugins/CoreServer/templates/SmtpServers/fields.inc b/app/plugins/CoreServer/templates/SmtpServers/fields.inc
@@ -25,11 +25,6 @@
  * @license       Apache License, Version 2.0 (http://www.apache.org/licenses/LICENSE-2.0)
  */
 
-// This view does currently not support read-only, and add is not used
-if($vv_action !== 'edit') {
-  return;
-}
-
 $fields = [
   'hostname',
   'port' => [

diff --git a/app/plugins/CoreServer/templates/SqlServers/fields.inc b/app/plugins/CoreServer/templates/SqlServers/fields.inc
@@ -25,11 +25,6 @@
  * @license       Apache License, Version 2.0 (http://www.apache.org/licenses/LICENSE-2.0)
  */
 
-// This view does currently not support read-only, and add is not used
-if($vv_action !== 'edit') {
-  return;
-}
-
 $fields = [
   'type',
   'hostname',

diff --git a/app/plugins/EnvSource/templates/EnvSourceCollectors/fields.inc b/app/plugins/EnvSource/templates/EnvSourceCollectors/fields.inc
@@ -25,11 +25,6 @@
  * @license       Apache License, Version 2.0 (http://www.apache.org/licenses/LICENSE-2.0)
  */
 
-// This view only supports edit
-if($vv_action !== 'edit') {
-  return;
-}  
-
 $fields = [
   'external_identity_source_id' => [
     'fieldLabel' => __d('env_source', 'controller.EnvSources', [1])

diff --git a/app/plugins/EnvSource/templates/EnvSources/fields.inc b/app/plugins/EnvSource/templates/EnvSources/fields.inc
@@ -25,11 +25,6 @@
  * @license       Apache License, Version 2.0 (http://www.apache.org/licenses/LICENSE-2.0)
  */
 
-// This view does not currently support read-only
-if($vv_action !== 'edit') {
-  return;
-}
-
 $fields = [
   // 'duplicate_mode',
   'sp_mode',

diff --git a/app/plugins/OrcidSource/templates/OrcidSourceCollectors/fields.inc b/app/plugins/OrcidSource/templates/OrcidSourceCollectors/fields.inc
@@ -25,11 +25,6 @@
  * @license       Apache License, Version 2.0 (http://www.apache.org/licenses/LICENSE-2.0)
  */
 
-// This view only supports edit
-if($vv_action !== 'edit') {
-  return;
-}
-
 $fields = [
   'external_identity_source_id' => [
     'fieldLabel' => __d('orcid_source', 'controller.OrcidSources', [1])

diff --git a/app/plugins/OrcidSource/templates/OrcidSources/fields.inc b/app/plugins/OrcidSource/templates/OrcidSources/fields.inc
@@ -27,11 +27,6 @@
 
 use OrcidSource\Lib\Enum\OrcidSourceScopeEnum;
 
-// This view does not currently support read-only
-if($vv_action !== 'edit') {
-  return;
-}
-
 $vv_inherited_scopes = OrcidSourceScopeEnum::DEFAULT_SCOPE;
 if (filter_var($vv_obj->scope_inherit, FILTER_VALIDATE_BOOLEAN)) {
   $vv_inherited_scopes = $vv_obj->server?->oauth2_server?->scope ?? OrcidSourceScopeEnum::DEFAULT_SCOPE;