Update the owner field help text at the model level #239

Signed-off-by: tdruez <tdruez@nexb.com>

Author: tdruez

component_catalog/forms.py

@@ -161,12 +161,7 @@ class Meta:
 
     def __init__(self, *args, **kwargs):
         super().__init__(*args, **kwargs)
-
         self.fields["owner"].dataspace = self.dataspace
-        self.fields["owner"].help_text = (
-            "Owner is the creator or maintainer of a Component, typically the current "
-            "copyright holder. This field is optional but recommended."
-        )
 
     def clean_packages_ids(self):
         packages_ids = self.cleaned_data.get("packages_ids")

component_catalog/migrations/0011_alter_component_owner.py

@@ -0,0 +1,20 @@
+# Generated by Django 5.1.6 on 2025-02-18 23:07
+
+import django.db.models.deletion
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('component_catalog', '0010_component_risk_score_package_risk_score'),
+        ('organization', '0001_initial'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='component',
+            name='owner',
+            field=models.ForeignKey(blank=True, help_text='Owner is the creator or maintainer of a component, typically the current copyright holder. This field is optional but recommended.', null=True, on_delete=django.db.models.deletion.PROTECT, to='organization.owner'),
+        ),
+    ]

component_catalog/models.py

@@ -709,17 +709,14 @@ class BaseComponentMixin(
             blank=True,
             on_delete=models.PROTECT,
             help_text=format_lazy(
-                "Owner is an optional field selected by the user to identify the original "
-                "creator (copyright holder) of the  {verbose_name}. "
-                "If this {verbose_name} is in its original, unmodified state, the {verbose_name}"
-                " owner is associated with the original author/publisher. "
-                "If this {verbose_name} has been copied and modified, "
-                "the {verbose_name}  owner should be the owner that has copied and "
-                "modified it.",
+                "Owner is the creator or maintainer of a {verbose_name}, typically the "
+                "current copyright holder. "
+                "This field is optional but recommended.",
                 verbose_name=_(verbose_name),
             ),
         )
 
+
         release_date = models.DateField(
             null=True,
             blank=True,

product_portfolio/forms.py

@@ -147,12 +147,7 @@ class Meta:
 
     def __init__(self, *args, **kwargs):
         super().__init__(*args, **kwargs)
-
         self.fields["owner"].dataspace = self.dataspace
-        self.fields["owner"].help_text = (
-            "Owner is the creator or maintainer of a Product, typically the current "
-            "copyright holder. This field is optional but recommended."
-        )
 
     def assign_object_perms(self, user):
         assign_perm("view_product", user, self.instance)

product_portfolio/migrations/0011_alter_product_owner.py

@@ -0,0 +1,20 @@
+# Generated by Django 5.1.6 on 2025-02-18 23:07
+
+import django.db.models.deletion
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('organization', '0001_initial'),
+        ('product_portfolio', '0010_productcomponent_weighted_risk_score_and_more'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='product',
+            name='owner',
+            field=models.ForeignKey(blank=True, help_text='Owner is the creator or maintainer of a product, typically the current copyright holder. This field is optional but recommended.', null=True, on_delete=django.db.models.deletion.PROTECT, to='organization.owner'),
+        ),
+    ]