diff --git a/js/src/components/Resource/FolderItem.vue b/js/src/components/Resource/FolderItem.vue index ecda49c88..e254c9224 100644 --- a/js/src/components/Resource/FolderItem.vue +++ b/js/src/components/Resource/FolderItem.vue @@ -29,7 +29,7 @@ class="actions" v-if="!inline" @delete="$emit('delete', resource.id)" - @move="$emit('move', resource.id)" + @move="$emit('move', resource)" @rename="$emit('rename', resource)" /> @@ -70,8 +70,6 @@ export default class FolderItem extends Mixins(ResourceMixin) { usernameWithDomain = usernameWithDomain; async onChange(evt: ChangeEvent): Promise { - console.log("into folder item"); - console.log(evt); if (evt.added && evt.added.element) { const movedResource = evt.added.element as IResource; const updatedResource = await this.moveResource(movedResource); diff --git a/js/src/components/Resource/ResourceItem.vue b/js/src/components/Resource/ResourceItem.vue index 4751e2a3b..c5865ed0d 100644 --- a/js/src/components/Resource/ResourceItem.vue +++ b/js/src/components/Resource/ResourceItem.vue @@ -29,7 +29,7 @@ class="actions" v-if="!inline" @delete="$emit('delete', resource.id)" - @move="$emit('move', resource.id)" + @move="$emit('move', resource)" @rename="$emit('rename', resource)" /> diff --git a/js/src/components/Resource/ResourceSelector.vue b/js/src/components/Resource/ResourceSelector.vue new file mode 100644 index 000000000..a16bb3be1 --- /dev/null +++ b/js/src/components/Resource/ResourceSelector.vue @@ -0,0 +1,129 @@ + + + diff --git a/js/src/graphql/resources.ts b/js/src/graphql/resources.ts index 65ea76422..5b1e90525 100644 --- a/js/src/graphql/resources.ts +++ b/js/src/graphql/resources.ts @@ -18,6 +18,7 @@ export const GET_RESOURCE = gql` summary url path + type metadata { ...ResourceMetadataBasicFields authorName @@ -28,6 +29,8 @@ export const GET_RESOURCE = gql` } parent { id + path + type } actor { id @@ -44,6 +47,11 @@ export const GET_RESOURCE = gql` type path resourceUrl + parent { + id + path + type + } metadata { ...ResourceMetadataBasicFields } @@ -112,7 +120,12 @@ export const UPDATE_RESOURCE = gql` summary url path + type resourceUrl + parent { + id + path + } } } `; diff --git a/js/src/i18n/en_US.json b/js/src/i18n/en_US.json index 951400e12..75cbc481a 100644 --- a/js/src/i18n/en_US.json +++ b/js/src/i18n/en_US.json @@ -696,5 +696,12 @@ "Can be an email or a link, or just plain text.": "Can be an email or a link, or just plain text.", "No profiles found": "No profiles found", "URL copied to clipboard": "URL copied to clipboard", - "Report #{reportNumber}": "Report #{reportNumber}" + "Report #{reportNumber}": "Report #{reportNumber}", + "Move \"{resourceName}\"": "Move \"{resourceName}\"", + "Parent folder": "Parent folder", + "(this folder)": "(this folder)", + "(this link)": "(this link)", + "Move resource to {folder}": "Move resource to {folder}", + "Create a folder": "Create a folder", + "No resources in this folder": "No resources in this folder" } diff --git a/js/src/i18n/fr_FR.json b/js/src/i18n/fr_FR.json index fc20e2fc5..450552732 100644 --- a/js/src/i18n/fr_FR.json +++ b/js/src/i18n/fr_FR.json @@ -696,5 +696,12 @@ "contact uninformed": "contact non renseigné", "Can be an email or a link, or just plain text.": "Peut être une adresse email ou bien un lien, ou alors du simple texte brut.", "URL copied to clipboard": "URL copiée dans le presse-papiers", - "Report #{reportNumber}": "Signalement #{reportNumber}" + "Report #{reportNumber}": "Signalement #{reportNumber}", + "Move \"{resourceName}\"": "Déplacer « {resourceName} »", + "Parent folder": "Dossier parent", + "(this folder)": "(ce dossier)", + "(this link)": "(ce lien)", + "Move resource to {folder}": "Déplacer la ressource dans {folder}", + "Create a folder": "Créer un dossier", + "No resources in this folder": "Aucune ressource dans ce dossier" } diff --git a/js/src/views/Resources/ResourceFolder.vue b/js/src/views/Resources/ResourceFolder.vue index eea87b067..28888591e 100644 --- a/js/src/views/Resources/ResourceFolder.vue +++ b/js/src/views/Resources/ResourceFolder.vue @@ -66,7 +66,7 @@
- +
{{ @@ -85,7 +85,12 @@
- +
@@ -100,18 +105,22 @@ v-if="localResource.type !== 'folder'" @delete="deleteResource" @rename="handleRename" + @move="handleMove" />
+
+

{{ $t("No resources in this folder") }}

+
@@ -126,6 +135,18 @@
+ +
+
+ +
+
+
@@ -190,9 +211,10 @@ import { import { CONFIG } from "../../graphql/config"; import { IConfig } from "../../types/config.model"; import ResourceMixin from "../../mixins/resource"; +import ResourceSelector from "../../components/Resource/ResourceSelector.vue"; @Component({ - components: { FolderItem, ResourceItem, Draggable }, + components: { FolderItem, ResourceItem, Draggable, ResourceSelector }, apollo: { resource: { query: GET_RESOURCE, @@ -253,6 +275,8 @@ export default class Resources extends Mixins(ResourceMixin) { createLinkResourceModal = false; + moveModal = false; + renameModal = false; groupObject: object = { @@ -332,6 +356,8 @@ export default class Resources extends Mixins(ResourceMixin) { createSentenceForType(type: string) { switch (type) { + case "folder": + return this.$t("Create a folder"); case "pad": return this.$t("Create a pad"); case "calc": @@ -347,7 +373,6 @@ export default class Resources extends Mixins(ResourceMixin) { } createResourceFromProvider(provider: IProvider) { - console.log(provider); this.newResource.resourceUrl = this.generateFullResourceUrl(provider); this.newResource.type = provider.software; this.createResourceModal = true; @@ -445,24 +470,100 @@ export default class Resources extends Mixins(ResourceMixin) { handleRename(resource: IResource) { this.renameModal = true; - this.updatedResource = resource; + this.updatedResource = Object.assign({}, resource); + } + + handleMove(resource: IResource) { + this.moveModal = true; + this.updatedResource = Object.assign({}, resource); + } + + async moveResource(resource: IResource, oldParent: IResource | undefined) { + const parentPath = oldParent && oldParent.path ? oldParent.path || "/" : "/"; + await this.updateResource(resource, parentPath); + this.moveModal = false; } async renameResource() { await this.updateResource(this.updatedResource); + this.renameModal = false; } - async updateResource(resource: IResource) { + async updateResource(resource: IResource, parentPath: string | null = null) { try { - if (!resource.parent) return; await this.$apollo.mutate<{ updateResource: IResource }>({ mutation: UPDATE_RESOURCE, variables: { id: resource.id, title: resource.title, - parentId: resource.parent.id, + parentId: resource.parent ? resource.parent.id : null, path: resource.path, }, + update: (store, { data }) => { + if (!data || data.updateResource == null || parentPath == null) return; + if (!this.resource.actor) return; + + console.log("Removing ressource from old parent"); + const oldParentCachedData = store.readQuery<{ resource: IResource }>({ + query: GET_RESOURCE, + variables: { + path: parentPath, + username: this.resource.actor.preferredUsername, + }, + }); + if (oldParentCachedData == null) return; + const { resource: oldParentCachedResource } = oldParentCachedData; + if (oldParentCachedResource == null) { + console.error("Cannot update resource cache, because of null value."); + return; + } + const resource: IResource = data.updateResource; + + oldParentCachedResource.children.elements = oldParentCachedResource.children.elements.filter( + (cachedResource) => cachedResource.id !== resource.id + ); + + store.writeQuery({ + query: GET_RESOURCE, + variables: { + path: parentPath, + username: this.resource.actor.preferredUsername, + }, + data: { oldParentCachedResource }, + }); + console.log("Finished removing ressource from old parent"); + + console.log("Adding resource to new parent"); + if (!resource.parent || !resource.parent.path) { + console.log("No cache found for new parent"); + return; + } + const newParentCachedData = store.readQuery<{ resource: IResource }>({ + query: GET_RESOURCE, + variables: { + path: resource.parent.path, + username: this.resource.actor.preferredUsername, + }, + }); + if (newParentCachedData == null) return; + const { resource: newParentCachedResource } = newParentCachedData; + if (newParentCachedResource == null) { + console.error("Cannot update resource cache, because of null value."); + return; + } + + newParentCachedResource.children.elements.push(resource); + + store.writeQuery({ + query: GET_RESOURCE, + variables: { + path: resource.parent.path, + username: this.resource.actor.preferredUsername, + }, + data: { newParentCachedResource }, + }); + console.log("Finished adding resource to new parent"); + }, }); } catch (e) { console.error(e); diff --git a/lib/graphql/schema.ex b/lib/graphql/schema.ex index 3f20d9f11..34fe0c309 100644 --- a/lib/graphql/schema.ex +++ b/lib/graphql/schema.ex @@ -5,7 +5,18 @@ defmodule Mobilizon.GraphQL.Schema do use Absinthe.Schema - alias Mobilizon.{Actors, Addresses, Conversations, Events, Media, Reports, Todos, Users} + alias Mobilizon.{ + Actors, + Addresses, + Conversations, + Events, + Media, + Reports, + Resources, + Todos, + Users + } + alias Mobilizon.Actors.{Actor, Follower, Member} alias Mobilizon.Conversations.Comment alias Mobilizon.Events.{Event, Participant} @@ -109,6 +120,7 @@ defmodule Mobilizon.GraphQL.Schema do |> Dataloader.add_source(Addresses, default_source) |> Dataloader.add_source(Media, default_source) |> Dataloader.add_source(Reports, default_source) + |> Dataloader.add_source(Resources, default_source) |> Dataloader.add_source(Todos, default_source) Map.put(ctx, :loader, loader) diff --git a/lib/graphql/schema/resource.ex b/lib/graphql/schema/resource.ex index f5a1e717b..6b91ba21a 100644 --- a/lib/graphql/schema/resource.ex +++ b/lib/graphql/schema/resource.ex @@ -4,6 +4,8 @@ defmodule Mobilizon.GraphQL.Schema.ResourceType do """ use Absinthe.Schema.Notation alias Mobilizon.GraphQL.Resolvers.Resource + alias Mobilizon.Resources + import Absinthe.Resolution.Helpers, only: [dataloader: 1] @desc "A resource" object :resource do @@ -19,7 +21,8 @@ defmodule Mobilizon.GraphQL.Schema.ResourceType do field(:updated_at, :naive_datetime, description: "The resource's last update date") field(:type, :string, description: "The resource's type (if it's a folder)") field(:path, :string, description: "The resource's path") - field(:parent, :resource, description: "The resource's parent") + + field(:parent, :resource, description: "The resource's parent", resolve: dataloader(Resources)) field :children, :paginated_resource_list do description("Children resources in folder") diff --git a/lib/service/rich_media/parsers/ogp.ex b/lib/service/rich_media/parsers/ogp.ex index 1630d3841..8be34d71d 100644 --- a/lib/service/rich_media/parsers/ogp.ex +++ b/lib/service/rich_media/parsers/ogp.ex @@ -30,7 +30,16 @@ defmodule Mobilizon.Service.RichMedia.Parsers.OGP do defp transform_tags(data) do data |> Map.put(:image_remote_url, Map.get(data, :image)) - |> Map.put(:width, Map.get(data, :"image:width")) - |> Map.put(:height, Map.get(data, :"image:height")) + |> Map.put(:width, get_integer_value(data, :"image:width")) + |> Map.put(:height, get_integer_value(data, :"image:height")) + end + + defp get_integer_value(data, key) do + with value <- Map.get(data, key), + {value, ""} <- Integer.parse(value) do + value + else + _ -> nil + end end end