Skip to content

Commit 53d5401

Browse files
Merge pull request #6219 from christianbeeznest/GH-4767-4
Link: Fix link form category selection and ensure HTTPS default in URL - refs #4767
2 parents 03897e7 + 5f31b7b commit 53d5401

File tree

2 files changed

+18
-11
lines changed

2 files changed

+18
-11
lines changed

Diff for: assets/vue/components/links/LinkForm.vue

+17-10
Original file line numberDiff line numberDiff line change
@@ -139,7 +139,7 @@ const resourceLinkList = ref(
139139
const categories = ref([])
140140
141141
const formData = reactive({
142-
url: "http://",
142+
url: "https://",
143143
title: "",
144144
description: "",
145145
category: null,
@@ -189,7 +189,7 @@ const fetchLink = async () => {
189189
}
190190
191191
if (response.category) {
192-
formData.category = parseInt(response.category["@id"].split("/").pop())
192+
formData.category = response.category
193193
}
194194
} catch (error) {
195195
console.error("Error fetching link:", error)
@@ -228,20 +228,27 @@ const submitForm = async () => {
228228
if (props.linkId) {
229229
await linkService.updateLink(props.linkId, postData)
230230
231-
const formDataImage = new FormData()
232-
formDataImage.append("removeImage", formData.removeImage ? "true" : "false")
231+
if (formData.showOnHomepage && (formData.removeImage || selectedFile.value instanceof File)) {
232+
const formDataImage = new FormData()
233+
formDataImage.append("removeImage", formData.removeImage ? "true" : "false")
233234
234-
if (selectedFile.value instanceof File) {
235-
formDataImage.append("customImage", selectedFile.value)
236-
}
235+
if (selectedFile.value instanceof File) {
236+
formDataImage.append("customImage", selectedFile.value)
237+
}
237238
238-
await linkService.uploadImage(props.linkId, formDataImage)
239+
await linkService.uploadImage(props.linkId, formDataImage)
240+
}
239241
} else {
240242
const newLink = await linkService.createLink(postData)
241243
242-
if (selectedFile.value instanceof File) {
244+
if (formData.showOnHomepage && (formData.removeImage || selectedFile.value instanceof File)) {
243245
const formDataImage = new FormData()
244-
formDataImage.append("customImage", selectedFile.value)
246+
formDataImage.append("removeImage", formData.removeImage ? "true" : "false")
247+
248+
if (selectedFile.value instanceof File) {
249+
formDataImage.append("customImage", selectedFile.value)
250+
}
251+
245252
await linkService.uploadImage(newLink.iid, formDataImage)
246253
}
247254
}

Diff for: assets/vue/views/links/LinksList.vue

+1-1
Original file line numberDiff line numberDiff line change
@@ -382,7 +382,7 @@ async function fetchLinks() {
382382
try {
383383
const data = await linkService.getLinks(params)
384384
linksWithoutCategory.value = data.linksWithoutCategory || []
385-
categories.value = data.categories || []
385+
categories.value = Object.values(data.categories || {})
386386
} catch (error) {
387387
console.error("Error fetching links:", error)
388388
notifications.showErrorNotification(t("Could not retrieve links"))

0 commit comments

Comments
 (0)