Upload location is not validated when config is saved

Created on 22 January 2025, 8 days ago

Problem/Motivation

Upload Location should be validated or checked on configuration save. It shouldn't be left to an upload form failure to notify of an issue with the directory.

Steps to reproduce

Create a new bulk upload configuration
Leave the Upload Location empty or put in an invalid value
Save
Try to use your new form - on submission you finally get an error about the upload location you specified

Proposed resolution

On configuration save, check that the directory exists and is writable.

Remaining tasks

User interface changes

API changes

Data model changes

🐛 Bug report

Status

Active

Version

3.0

Component

Code

Created by

🇬🇧United Kingdom amoebanath

Live updates comments and jobs are added and updated live.

Merge Requests

!19Upload location is not validated when config is saved
Open
Unnamed author
updated 8 days ago

Comments & Activities

Issue created by @amoebanath
Comment 8 days ago →
nidhish
@amoebanath I have hereby attached the patch for validation of upload location, if it is empty. I could not understand the concept of individual value since, it is a temporary location. Kindly explain an invalid state so that I can create a MR fixing that as well.
Comment 8 days ago →
nidhish
Comment 8 days ago →
🇬🇧United Kingdom amoebanath
I've opened an issue fork that covers the scenarios I thought should reasonably be covered
Merge request !19Issue #3501365 Require and validate the temporary upload location → (Open) created by Unnamed author

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024