Ticket #887 (closed Defect: fixed)

Opened 5 years ago

Last modified 5 years ago

dm4-csv: upgrade to 4.8

Reported by: Malte Owned by: Malte
Priority: Major Milestone: Release 4.8
Component: 3rd Party Plugins Version: 4.7
Keywords: Cc: jri, dgf, JuergeN
Complexity: 3 Area:
Module:

Description

No release for 4.7 will be available for dm4-csv because of #883, using the file upload_dialog wont work for files trying to be uploaded to "/" (the root of the current filerepo).

The Background here is, the custom "csv" folder must and will be dropped because of:
(1) the "csv" folder has no meaning to the user, it is an automatically created folder and thus it restricts the freedom of the user to keep a filerepo organized up to their needs.

(2) with the per_workspace setting it becomes impossibly to initiate a custom folder (e.g. during install, or in a migration since) as there are not all workspaces present yet, thus "on the fly" folder creation is the only option (as it is working well for the dm4-images and the dm4-import-export modules) but for the dm4-csv plugin developer there is no other option to create it "on the fly" than replacing the dm4-files plugin file upload mechanism through its own implementation.

Change History

comment:1 Changed 5 years ago by Malte

on (2): which i don't regard as a desirable option (if it is just for creating some mechanic custom folder)

comment:2 in reply to: ↑ description Changed 5 years ago by jri

Replying to Malte:

No release for 4.7 will be available for dm4-csv because of #883, using the file upload_dialog wont work for files trying to be uploaded to "/" (the root of the current filerepo).

From my point of view #883 is not relevant here. The upload request URL is independent from where the uploaded file is stored, if at all. See ticket:888#comment:1

comment:3 Changed 5 years ago by jri

See #888.
Does this work for you?

comment:4 Changed 5 years ago by jri

See also this posting:
http://lists.deepamehta.de/pipermail/devel-lists.deepamehta.de/2015-November/000605.html
The solution might be #884, which is meanwhile realized.

comment:5 follow-up: ↓ 6 Changed 5 years ago by Malte

Yes, thanks to #888 i got to know that i now need to use "encodedURIComponent(path)" when passing a repo-path to the files service endpoint.

This also means that a release for 4.7 could be made yesterday:
http://download.deepamehta.de/dm47-deepamehta-csv-0.0.5.jar

Nonetheless i would want to leave this ticket open to cover the task of adapting the dm4-csv plugin to the upcoming 4.7.1 release.

comment:6 in reply to: ↑ 5 Changed 5 years ago by jri

Replying to Malte:

This also means that a release for 4.7 could be made yesterday:
http://download.deepamehta.de/dm47-deepamehta-csv-0.0.5.jar

Great :-)

Nonetheless i would want to leave this ticket open to cover the task of adapting the dm4-csv plugin to the upcoming 4.7.1 release.

OK

comment:7 Changed 5 years ago by Malte

  • Summary changed from dm4-csv: upgrading to 4.7.1 to dm4-csv: upgrade to 4.8

comment:8 Changed 5 years ago by Malte

  • Status changed from new to closed
  • Resolution set to fixed

No improvements usability or functionality wise here (for a detailed vision see #233) but:
https://download.deepamehta.de/dm48-deepamehta-csv-0.0.6.jar

Note: See TracTickets for help on using tickets.