diff --git a/backend/toolshed/admin.py b/backend/toolshed/admin.py index cea1e9b..d813ac7 100644 --- a/backend/toolshed/admin.py +++ b/backend/toolshed/admin.py @@ -18,11 +18,10 @@ class ItemPropertyInline(admin.TabularInline): class InventoryItemAdmin(admin.ModelAdmin): - list_display = ('name', 'description', 'category', 'availability_policy', 'visibility_policy', 'owned_quantity', - 'owner', 'owner_group', 'storage_location', 'get_tags', 'get_properties') - search_fields = ('name', 'description', 'category__name', 'availability_policy', 'visibility_policy', - 'owner__username', 'owner_group__name', 'storage_location__name', 'tags__name', - 'itemproperty__property__name') + list_display = ('name', 'description', 'category', 'availability_policy', 'owned_quantity', 'owner', + 'owner_group', 'storage_location', 'get_tags', 'get_properties') + search_fields = ('name', 'description', 'category__name', 'availability_policy', 'owner__username', + 'owner_group__name', 'storage_location__name', 'tags__name', 'itemproperty__property__name') inlines = (ItemTagInline, ItemPropertyInline) def get_queryset(self, request): @@ -65,10 +64,9 @@ admin.site.register(Category, CategoryAdmin) class StorageLocationAdmin(admin.ModelAdmin): - list_display = ('name', 'description', 'category', 'parent', 'owner', 'visibility_policy') - search_fields = ('name', 'description', 'category__name', 'parent__name', 'owner__username', - 'visibility_policy') - list_filter = ('category', 'owner', 'visibility_policy') + list_display = ('name', 'description', 'category', 'parent', 'owner') + search_fields = ('name', 'description', 'category__name', 'parent__name', 'owner__username') + list_filter = ('category', 'owner') admin.site.register(StorageLocation, StorageLocationAdmin) diff --git a/backend/toolshed/api/idmap.py b/backend/toolshed/api/idmap.py index d6050f9..a6451f1 100644 --- a/backend/toolshed/api/idmap.py +++ b/backend/toolshed/api/idmap.py @@ -43,8 +43,7 @@ def resolve_short_id(request, kind, owner_id, local_id): except model.DoesNotExist: return Response(status=404) is_owner = request.user.user.filter(pk=owner.pk).exists() - if not is_owner and (getattr(obj, 'availability_policy', 'share') == 'private' - or obj.visibility_policy == 'private'): + if getattr(obj, 'availability_policy', 'share') == 'private' and not is_owner: return Response(status=403) return Response({'handle': f'{owner.username}@{owner.domain}', 'id': obj.id}) if kind in ('group_item', 'group_storage_location'): diff --git a/backend/toolshed/api/info.py b/backend/toolshed/api/info.py index 8d9e7dd..6fc5036 100644 --- a/backend/toolshed/api/info.py +++ b/backend/toolshed/api/info.py @@ -46,13 +46,6 @@ def list_availability_policies(request, format=None): return Response(InventoryItem.AVAILABILITY_POLICY_CHOICES) -@api_view(['GET']) -@permission_classes([IsAuthenticated]) -@authentication_classes([SignatureAuthentication]) -def list_visibility_policies(request, format=None): - return Response(InventoryItem.VISIBILITY_POLICY_CHOICES) - - @api_view(['GET']) @permission_classes([IsAuthenticated]) @authentication_classes([SignatureAuthentication]) @@ -61,17 +54,15 @@ def combined_info(request, format=None): properties = PropertySerializer(Property.objects.all(), many=True).data categories = [str(category) for category in Category.objects.all()] policies = InventoryItem.AVAILABILITY_POLICY_CHOICES - visibility_policies = InventoryItem.VISIBILITY_POLICY_CHOICES domains = [domain.name for domain in Domain.objects.filter(open_registration=True)] return Response( - {'tags': tags, 'properties': properties, 'availability_policies': policies, - 'visibility_policies': visibility_policies, 'categories': categories, 'domains': domains}) + {'tags': tags, 'properties': properties, 'availability_policies': policies, 'categories': categories, + 'domains': domains}) urlpatterns = [ path('availability_policies/', list_availability_policies, name='availability_policies'), - path('visibility_policies/', list_visibility_policies, name='visibility_policies'), path('properties/', list_properties, name='propertylist'), path('categories/', list_categories, name='categorylist'), path('domains/', list_domains, name='domainlist'), diff --git a/backend/toolshed/api/inventory.py b/backend/toolshed/api/inventory.py index 7cc61d1..15b7eac 100644 --- a/backend/toolshed/api/inventory.py +++ b/backend/toolshed/api/inventory.py @@ -25,7 +25,7 @@ def inventory_items(identity): friend_user = friend.user.first() if friend_user: for item in friend_user.inventory_items.all(): - if item.availability_policy != 'private' and item.visibility_policy != 'private': + if item.availability_policy != 'private': yield item @@ -55,7 +55,7 @@ class InventoryItemViewSet(viewsets.ModelViewSet): return InventoryItem.objects.none() queryset = InventoryItem.objects.filter(owner=owner_user) if not identity.user.filter(pk=owner_user.pk).exists(): - queryset = queryset.exclude(availability_policy='private').exclude(visibility_policy='private') + queryset = queryset.exclude(availability_policy='private') else: return InventoryItem.objects.none() # InventoryItemSerializer touches owner/owner_group/category/storage_location (FKs) and @@ -132,7 +132,7 @@ class StorageLocationViewSet(viewsets.ModelViewSet): lookup_url_kwarg = 'pk' def get_queryset(self): - # See docs/implementation.md#owner-handle-scoped-routes; unlike items, StorageLocation has no availability_policy, but visibility_policy='private' still hides a location from friends. + # See docs/implementation.md#owner-handle-scoped-routes; unlike items, StorageLocation has no availability_policy, so a friend sees all of a user's locations. if type(self.request.user) != KnownIdentity: return StorageLocation.objects.none() identity = self.request.user @@ -147,10 +147,7 @@ class StorageLocationViewSet(viewsets.ModelViewSet): if owner_user: if owner_user not in identity.friends_or_self(): return StorageLocation.objects.none() - queryset = StorageLocation.objects.filter(owner=owner_user) - if not identity.user.filter(pk=owner_user.pk).exists(): - queryset = queryset.exclude(visibility_policy='private') - return queryset + return StorageLocation.objects.filter(owner=owner_user) return StorageLocation.objects.none() def perform_create(self, serializer): diff --git a/backend/toolshed/migrations/0025_inventoryitem_visibility_policy_and_more.py b/backend/toolshed/migrations/0025_inventoryitem_visibility_policy_and_more.py deleted file mode 100644 index e427932..0000000 --- a/backend/toolshed/migrations/0025_inventoryitem_visibility_policy_and_more.py +++ /dev/null @@ -1,23 +0,0 @@ -# Generated by Django 4.2.2 on 2026-08-31 23:55 - -from django.db import migrations, models - - -class Migration(migrations.Migration): - - dependencies = [ - ('toolshed', '0024_workflowinstance_current_step'), - ] - - operations = [ - migrations.AddField( - model_name='inventoryitem', - name='visibility_policy', - field=models.CharField(choices=[('public', 'Public'), ('friends', 'Friends'), ('private', 'Private')], default='friends', max_length=20), - ), - migrations.AddField( - model_name='storagelocation', - name='visibility_policy', - field=models.CharField(choices=[('public', 'Public'), ('friends', 'Friends'), ('private', 'Private')], default='friends', max_length=20), - ), - ] diff --git a/backend/toolshed/models.py b/backend/toolshed/models.py index 6113f6a..65de960 100644 --- a/backend/toolshed/models.py +++ b/backend/toolshed/models.py @@ -107,13 +107,6 @@ class OwnerItemSequence(models.Model): return seq.last_id -VISIBILITY_POLICY_CHOICES = ( - ('public', 'Public'), - ('friends', 'Friends'), - ('private', 'Private'), -) - - class InventoryItem(SoftDeleteModel): AVAILABILITY_POLICY_CHOICES = ( ('sell', 'Sell'), @@ -122,7 +115,6 @@ class InventoryItem(SoftDeleteModel): ('share', 'Share'), ('private', 'Private'), ) - VISIBILITY_POLICY_CHOICES = VISIBILITY_POLICY_CHOICES internal_id = models.AutoField(primary_key=True) # Externally visible id, sequential/gapless within owner/owner_group's own items (see @@ -133,7 +125,6 @@ class InventoryItem(SoftDeleteModel): description = models.TextField(null=True, blank=True) category = models.ForeignKey(Category, on_delete=models.CASCADE, null=True, related_name='inventory_items') availability_policy = models.CharField(max_length=20, choices=AVAILABILITY_POLICY_CHOICES, default='private') - visibility_policy = models.CharField(max_length=20, choices=VISIBILITY_POLICY_CHOICES, default='private') owned_quantity = models.IntegerField(default=1, validators=[MinValueValidator(0)]) owner = models.ForeignKey(ToolshedUser, on_delete=models.CASCADE, null=True, blank=True, related_name='inventory_items') @@ -204,8 +195,6 @@ class OwnerStorageLocationSequence(models.Model): class StorageLocation(models.Model): - VISIBILITY_POLICY_CHOICES = VISIBILITY_POLICY_CHOICES - internal_id = models.AutoField(primary_key=True) # Externally visible id, sequential/gapless within the owner/owner_group's own locations (see # OwnerStorageLocationSequence), never internal_id; always allocate via create_for_owner, not @@ -220,7 +209,6 @@ class StorageLocation(models.Model): related_name='storage_locations') owner_group = models.ForeignKey(Group, on_delete=models.CASCADE, null=True, blank=True, related_name='storage_locations') - visibility_policy = models.CharField(max_length=20, choices=VISIBILITY_POLICY_CHOICES, default='private') class Meta: constraints = [ diff --git a/backend/toolshed/offlinedata.py b/backend/toolshed/offlinedata.py index 0f93f1e..ee9f383 100644 --- a/backend/toolshed/offlinedata.py +++ b/backend/toolshed/offlinedata.py @@ -24,7 +24,6 @@ def inventory_rows(user): 'description': item.description or '', 'category': item.category.get_handle() if item.category else '', 'availability_policy': item.availability_policy, - 'visibility_policy': item.visibility_policy, 'owned_quantity': item.owned_quantity, 'storage_location': str(item.storage_location) if item.storage_location else '', 'tags': ', '.join(tag.get_handle() for tag in item.tags.all()), @@ -68,7 +67,6 @@ def location_rows(user): 'category': str(location.category) if location.category else '', 'parent': str(location.parent) if location.parent else '', 'path': location_path(location), - 'visibility_policy': location.visibility_policy, } @@ -344,7 +342,6 @@ def import_locations(user, data): defaults = { 'description': row.get('description', '') or '', 'category': category, - 'visibility_policy': row.get('visibility_policy') or 'private', } try: location = StorageLocation.objects.get(owner=user, name=name, parent=parent) @@ -554,7 +551,6 @@ def import_inventory(user, data, available_files): description=row.get('description', '') or '', category=category, availability_policy=row.get('availability_policy') or 'private', - visibility_policy=row.get('visibility_policy') or 'private', owned_quantity=owned_quantity, storage_location=storage_location, ) diff --git a/backend/toolshed/serializers.py b/backend/toolshed/serializers.py index af21446..07f60b9 100644 --- a/backend/toolshed/serializers.py +++ b/backend/toolshed/serializers.py @@ -211,8 +211,7 @@ class StorageLocationSerializer(serializers.ModelSerializer): class Meta: model = StorageLocation - fields = ['id', 'name', 'description', 'path', 'category', 'owner', 'owner_group', 'parent', - 'visibility_policy'] + fields = ['id', 'name', 'description', 'path', 'category', 'owner', 'owner_group', 'parent'] read_only_fields = ['id', 'path'] @staticmethod @@ -259,8 +258,7 @@ class InventoryItemSerializer(serializers.ModelSerializer): class Meta: model = InventoryItem fields = ['id', 'name', 'description', 'owner', 'owner_group', 'category', 'availability_policy', - 'visibility_policy', 'owned_quantity', 'tags', 'tags_input', 'properties', 'files', - 'storage_location'] + 'owned_quantity', 'tags', 'tags_input', 'properties', 'files', 'storage_location'] read_only_fields = ['id'] def get_tags(self, obj): diff --git a/backend/toolshed/tests/fixtures.py b/backend/toolshed/tests/fixtures.py index 9f4e647..69c2e42 100644 --- a/backend/toolshed/tests/fixtures.py +++ b/backend/toolshed/tests/fixtures.py @@ -37,10 +37,10 @@ class InventoryTestMixin(CategoryTestMixin, TagTestMixin, PropertyTestMixin): self.f['item1'] = InventoryItem.create_for_owner( owner=self.f['local_user1'], owned_quantity=1, name='test1', description='test', category=self.f['cat1'], - availability_policy='friends', visibility_policy='friends') + availability_policy='friends') self.f['item2'] = InventoryItem.create_for_owner( owner=self.f['local_user1'], owned_quantity=1, name='test2', description='test2', category=self.f['cat1'], - availability_policy='friends', visibility_policy='friends') + availability_policy='friends') self.f['item2'].tags.add(self.f['tag1'], through_defaults={}) self.f['item2'].tags.add(self.f['tag2'], through_defaults={}) ItemProperty.objects.create(inventory_item=self.f['item2'], property=self.f['prop1'], value='value1').save() @@ -49,14 +49,12 @@ class InventoryTestMixin(CategoryTestMixin, TagTestMixin, PropertyTestMixin): class LocationTestMixin: def prepare_locations(self): - self.f['loc1'] = StorageLocation.create_for_owner(name='loc1', owner=self.f['local_user1'], - visibility_policy='friends') + self.f['loc1'] = StorageLocation.create_for_owner(name='loc1', owner=self.f['local_user1']) self.f['loc2'] = StorageLocation.create_for_owner(name='loc2', owner=self.f['local_user1'], - category=self.f['cat1'], visibility_policy='friends') - self.f['loc3'] = StorageLocation.create_for_owner(name='loc3', owner=self.f['local_user1'], parent=self.f['loc1'], - visibility_policy='friends') + category=self.f['cat1']) + self.f['loc3'] = StorageLocation.create_for_owner(name='loc3', owner=self.f['local_user1'], parent=self.f['loc1']) self.f['loc4'] = StorageLocation.create_for_owner(name='loc4', owner=self.f['local_user1'], parent=self.f['loc1'], - category=self.f['cat1'], visibility_policy='friends') + category=self.f['cat1']) class WorkflowTestMixin: diff --git a/backend/toolshed/tests/test_api.py b/backend/toolshed/tests/test_api.py index e8b41a1..69c25df 100644 --- a/backend/toolshed/tests/test_api.py +++ b/backend/toolshed/tests/test_api.py @@ -46,15 +46,6 @@ class CombinedApiTestCase(UserTestMixin, CategoryTestMixin, TagTestMixin, Proper self.assertEqual(response.json(), [['sell', 'Sell'], ['rent', 'Rent'], ['lend', 'Lend'], ['share', 'Share'], ['private', 'Private']]) - def test_visibility_policy_api_anonymous(self): - response = anonymous_client.get('/api/v1/visibility_policies/') - self.assertEqual(response.status_code, 403) - - def test_visibility_policy_api(self): - response = client.get('/api/v1/visibility_policies/', self.f['local_user1']) - self.assertEqual(response.status_code, 200) - self.assertEqual(response.json(), [['public', 'Public'], ['friends', 'Friends'], ['private', 'Private']]) - def test_combined_api_anonymous(self): response = anonymous_client.get('/api/v1/info/') self.assertEqual(response.status_code, 403) @@ -64,8 +55,6 @@ class CombinedApiTestCase(UserTestMixin, CategoryTestMixin, TagTestMixin, Proper self.assertEqual(response.status_code, 200) self.assertEqual(response.json()['availability_policies'], [['sell', 'Sell'], ['rent', 'Rent'], ['lend', 'Lend'], ['share', 'Share'], ['private', 'Private']]) - self.assertEqual(response.json()['visibility_policies'], - [['public', 'Public'], ['friends', 'Friends'], ['private', 'Private']]) self.assertEqual(response.json()['categories'], ['cat1', 'cat2', 'cat3', 'cat1/subcat1', 'cat1/subcat2', 'cat1/subcat1/subcat1', 'cat1/subcat1/subcat2']) diff --git a/backend/toolshed/tests/test_idmap.py b/backend/toolshed/tests/test_idmap.py index d486f84..c73335a 100644 --- a/backend/toolshed/tests/test_idmap.py +++ b/backend/toolshed/tests/test_idmap.py @@ -89,17 +89,6 @@ class ResolveShortIdApiTestCase(UserTestMixin, InventoryTestMixin, GroupTestMixi self.f['local_user1']) self.assertEqual(reply.status_code, 200) - def test_resolve_item_visibility_private_not_owner(self): - private_item = InventoryItem.create_for_owner( - owner=self.f['local_user1'], owned_quantity=1, name='secret', availability_policy='friends', - visibility_policy='private') - reply = client.get(f'/api/v1/resolve_short_id/item/{self.owner_id}/{private_item.id}/', - self.f['local_user2']) - self.assertEqual(reply.status_code, 403) - reply = client.get(f'/api/v1/resolve_short_id/item/{self.owner_id}/{private_item.id}/', - self.f['local_user1']) - self.assertEqual(reply.status_code, 200) - def test_resolve_item_unknown_owner(self): reply = client.get('/api/v1/resolve_short_id/item/999999/1/', self.f['local_user2']) self.assertEqual(reply.status_code, 404) @@ -120,16 +109,6 @@ class ResolveShortIdApiTestCase(UserTestMixin, InventoryTestMixin, GroupTestMixi self.assertEqual(reply.status_code, 200) self.assertEqual(reply.json(), {'handle': 'testuser1@example.com', 'id': self.f['loc1'].id}) - def test_resolve_storage_location_visibility_private_not_owner(self): - private_location = StorageLocation.create_for_owner( - owner=self.f['local_user1'], name='secret-loc', visibility_policy='private') - reply = client.get(f'/api/v1/resolve_short_id/storage_location/{self.owner_id}/{private_location.id}/', - self.f['local_user2']) - self.assertEqual(reply.status_code, 403) - reply = client.get(f'/api/v1/resolve_short_id/storage_location/{self.owner_id}/{private_location.id}/', - self.f['local_user1']) - self.assertEqual(reply.status_code, 200) - def test_resolve_storage_location_unknown_local_id(self): reply = client.get(f'/api/v1/resolve_short_id/storage_location/{self.owner_id}/999999/', self.f['local_user1']) diff --git a/backend/toolshed/tests/test_inventory.py b/backend/toolshed/tests/test_inventory.py index 8a08b49..162d7c5 100644 --- a/backend/toolshed/tests/test_inventory.py +++ b/backend/toolshed/tests/test_inventory.py @@ -205,15 +205,6 @@ class InventoryApiTestCase(UserTestMixin, InventoryTestMixin, ToolshedTestCase): self.assertEqual(reply.json()[0]['name'], 'test1') self.assertEqual(reply.json()[1]['name'], 'test2') - def test_search_items_excludes_visibility_private(self): - InventoryItem.create_for_owner( - owner=self.f['local_user1'], owned_quantity=1, name='test-secret', availability_policy='friends', - visibility_policy='private') - reply = client.get('/api/v1/search/?query=test', self.f['local_user2']) - self.assertEqual(reply.status_code, 200) - self.assertEqual(len(reply.json()), 2) - self.assertNotIn('test-secret', [item['name'] for item in reply.json()]) - def test_search_items_fail(self): reply = client.get('/api/v1/search/', self.f['local_user1']) self.assertEqual(reply.status_code, 400) @@ -255,17 +246,6 @@ class InventoryApiTestCase(UserTestMixin, InventoryTestMixin, ToolshedTestCase): self.f['local_user1']) self.assertEqual(reply.status_code, 200) - def test_get_shared_item_visibility_private(self): - private_item = InventoryItem.create_for_owner( - owner=self.f['local_user1'], owned_quantity=1, name='secret', availability_policy='friends', - visibility_policy='private') - reply = client.get('/api/v1/inventory_items/testuser1@example.com/' + str(private_item.id) + '/', - self.f['local_user2']) - self.assertEqual(reply.status_code, 404) - reply = client.get('/api/v1/inventory_items/testuser1@example.com/' + str(private_item.id) + '/', - self.f['local_user1']) - self.assertEqual(reply.status_code, 200) - def test_get_shared_item_unknown_handle(self): reply = client.get('/api/v1/inventory_items/nobody@example.com/' + str(self.f['item1'].id) + '/', self.f['local_user2']) diff --git a/backend/toolshed/tests/test_locations.py b/backend/toolshed/tests/test_locations.py index 5c1ab75..c6d84b4 100644 --- a/backend/toolshed/tests/test_locations.py +++ b/backend/toolshed/tests/test_locations.py @@ -128,23 +128,6 @@ class LocationApiTestCase(UserTestMixin, InventoryTestMixin, LocationTestMixin, self.assertEqual(reply.status_code, 200) self.assertEqual(len(reply.json()), 0) - def test_get_shared_location_visibility_private(self): - private_location = StorageLocation.create_for_owner( - owner=self.f['local_user1'], name='secret-loc', visibility_policy='private') - reply = client.get( - '/api/v1/storage_locations/{}/{}/'.format(self.own_handle, private_location.id), self.f['local_user2']) - self.assertEqual(reply.status_code, 404) - reply = client.get( - '/api/v1/storage_locations/{}/{}/'.format(self.own_handle, private_location.id), self.f['local_user1']) - self.assertEqual(reply.status_code, 200) - - def test_list_locations_excludes_visibility_private_for_friend(self): - StorageLocation.create_for_owner(owner=self.f['local_user1'], name='secret-loc', visibility_policy='private') - reply = client.get('/api/v1/storage_locations/{}/'.format(self.own_handle), self.f['local_user2']) - self.assertEqual(reply.status_code, 200) - self.assertEqual(len(reply.json()), 4) - self.assertNotIn('secret-loc', [loc['name'] for loc in reply.json()]) - def test_cannot_delete_other_users_location(self): # local_user2 is a friend of local_user1 (see prepare_inventory), so local_user1's own # handle resolves and the location is visible - friends can read but never write, so this diff --git a/backend/toolshed/tests/test_offlinedata.py b/backend/toolshed/tests/test_offlinedata.py index c558fa4..8a5a732 100644 --- a/backend/toolshed/tests/test_offlinedata.py +++ b/backend/toolshed/tests/test_offlinedata.py @@ -233,7 +233,7 @@ class ExportImportApiRoundTripTestCase(UserTestMixin, CategoryTestMixin, TagTest item = InventoryItem.create_for_owner( owner=self.f['local_user1'], name='drill', description='cordless drill', - category=self.f['cat1'], availability_policy='friends', visibility_policy='public', owned_quantity=2) + category=self.f['cat1'], availability_policy='friends', owned_quantity=2) item.tags.add(self.f['tag1'], self.f['tag2'], through_defaults={}) ItemProperty.objects.create(inventory_item=item, property=self.f['prop1'], value='10cm, 20cm') ItemProperty.objects.create(inventory_item=item, property=self.f['prop2'], value='a=b') @@ -251,7 +251,6 @@ class ExportImportApiRoundTripTestCase(UserTestMixin, CategoryTestMixin, TagTest new_item = InventoryItem.objects.get(owner=self.f['local_user2'], name='drill') self.assertEqual(new_item.category, self.f['cat1']) - self.assertEqual(new_item.visibility_policy, 'public') self.assertEqual(sorted(t.name for t in new_item.tags.all()), ['tag1', 'tag2']) values = {ip.property.name: ip.value for ip in new_item.itemproperty_set.select_related('property')} diff --git a/frontend/src/components/workflow/workflows/FotoFirstBulkImportWorkflow.vue b/frontend/src/components/workflow/workflows/FotoFirstBulkImportWorkflow.vue index cc20c9c..1998a36 100644 --- a/frontend/src/components/workflow/workflows/FotoFirstBulkImportWorkflow.vue +++ b/frontend/src/components/workflow/workflows/FotoFirstBulkImportWorkflow.vue @@ -98,7 +98,6 @@ @@ -188,7 +187,6 @@ @@ -295,7 +293,6 @@ const PRESET_FIELD_DEFS = [ {key: 'owner_group', label: 'Owner', type: 'owner_group'}, {key: 'owned_quantity', label: 'Quantity', type: 'number'}, {key: 'availability_policy', label: 'Availability Policy', type: 'availability_policy'}, - {key: 'visibility_policy', label: 'Visibility Policy', type: 'visibility_policy'}, {key: 'storage_location', label: 'Storage Location', type: 'storage_location'} ]; @@ -310,7 +307,6 @@ function defaultValueFor(key) { case 'owned_quantity': return 1; case 'availability_policy': - case 'visibility_policy': return 'private'; default: return ''; @@ -394,7 +390,7 @@ export default { } }, computed: { - ...mapState(['user', 'availability_policies', 'visibility_policies', 'storage_locations', 'groups', 'groupMemberships']), + ...mapState(['user', 'availability_policies', 'storage_locations', 'groups', 'groupMemberships']), ownerGroups() { const hostedHandles = new Set(this.groups.map(group => group.handle)); const foreign = this.groupMemberships.filter(m => !hostedHandles.has(m.handle)); @@ -623,7 +619,6 @@ export default { properties: details.properties, owned_quantity: details.owned_quantity, availability_policy: details.availability_policy, - visibility_policy: details.visibility_policy, storage_location: details.storage_location, owner_group: details.owner_group }); diff --git a/frontend/src/components/workflow/workflows/FotoFirstFieldInput.vue b/frontend/src/components/workflow/workflows/FotoFirstFieldInput.vue index 206c9a4..3f8291e 100644 --- a/frontend/src/components/workflow/workflows/FotoFirstFieldInput.vue +++ b/frontend/src/components/workflow/workflows/FotoFirstFieldInput.vue @@ -20,12 +20,6 @@ - - -
- - -
-
- - -
-
- - -
-
- - -