Port upload path to 6.x

lewmich - March 22, 2008 - 22:42
Project:Upload path
Version:5.x-1.x-dev
Component:Code
Category:task
Priority:normal
Assigned:Unassigned
Status:closed
Description

Is anyone working on 6.x version of Upload path module?

Or are there any alternatives (different modules) which allow uploading files into subdirectories?

#1

encho - April 2, 2008 - 14:14

I am interested in this as well. Looks like development is on hold.

#2

maxiorel - April 4, 2008 - 10:10

I'm interested too. Maybe this (http://drupal.org/node/242582) can help us.

#3

pobster - April 4, 2008 - 11:15
Status:active» needs review

Try this.

Pobster

AttachmentSize
uploadpath.tar_.gz 7.36 KB

#4

sonnen - April 9, 2008 - 20:16

Attached is a unified patch that will take a stock distribution of Upload Path 5.x-1.x-dev up to a fully 6x compatible version. It includes a correction of the hook_nodeapi bug noted by mansion, but with 6x compatible code. (http://drupal.org/node/191571)

AttachmentSize
uploadpath_5x_to_6x.patch 4.23 KB

#5

greggles - April 29, 2008 - 20:49
Title:Anyone working on 6.x version?» Port upload path to 6.x

slightly better title.

#6

ansorg - May 9, 2008 - 20:05

Hi,

I think this patch contains an error when it tries to move the file.

<?php
             
if (file_move($file['filepath'], $dest, FILE_EXISTS_RENAME)) {
?>
this line tries to move the file from old location to the new "dest". But $dest does not contain the full path

I changed that to include the default "files" path in front of the newly assembled $dest filename

<?php
             
if (file_move($file['filepath'], file_directory_path() . '/' . $dest, FILE_EXISTS_RENAME)) {
?>

Jens

#7

ArjanLikesDrupal - June 17, 2008 - 23:34

From reading these threads it seems Upload path with patches is working -more or less- with D6. Will there be a D6 release anytime soon?

#8

ArjanLikesDrupal - June 22, 2008 - 14:19

Hi, I applied the patch and it works on my D6 site! Many thanks, I think this module provides a quite vital functionality if one wants to keep files organized on server.

One note: the current tarball is a 2008-May-05 release and rolling the patch gives some errors:

patching file uploadpath.info
Hunk #1 FAILED at 1.
1 out of 1 hunk FAILED -- saving rejects to file uploadpath.info.rej
patching file uploadpath.module
Hunk #2 FAILED at 56.

I don't know how to fix the patch, I just deleted the old parts and pasted the new ones in the uploadpath.info and uploadpath.module files manually. Maybe someone can fix this for current release?

#9

coupet - June 23, 2008 - 18:19

Can we have a patch for current dev version or final working d6 package?

#10

ArjanLikesDrupal - June 24, 2008 - 15:17

Here are the 6.x patched .module and .info files.
edit: replaced with the patch: here is the patch, modified to be used on the 2008-May-05 release of UploadPath.

AttachmentSize
uploadpath_5x_to_6x_for_2008-May-05-release.patch 4.76 KB

#11

greggles - June 24, 2008 - 14:26

@ArjanLikesDrupal - ideally patches should be provided as actual patches - (see http://drupal.org/patch/create ).

Also, this module is currently "Unmaintained" and will need someone to take it over from Crell. Perhaps someone on this issue can do that? If so, just 1) get a CVS account (if you don't have one point to this issue as the reason 2) contact Crell and/or file an issue in the webmasters queue about this module.

#12

ArjanLikesDrupal - June 24, 2008 - 18:46

Done. I had never used/created patches before so didn't know how to do so, but it wasn't hard.
As for the unmaintained status: agree, a 6.x release would also be nice. I'm afraid I'm of no use here, being at the start of my Drupal learning curve ;)

#13

greggles - June 24, 2008 - 16:57

Well, everyone has to start somewhere. I hope you will consider becoming the maintainer ( http://drupal.org/handbook/cvs is a good section to read to understand what is involved).

#14

coupet - June 24, 2008 - 18:07

Thanks ArjanLikesDrupal.

#15

sprugman - July 1, 2008 - 18:09

subscribing

#16

Sirn - September 3, 2008 - 20:09

Updated patch for 2008-Jul-24 release.

AttachmentSize
uploadpath_5x_to_6x_for_2008-Jul-24-release.diff 5.38 KB

#17

davidwhthomas - September 4, 2008 - 06:00

Thanks for the interest, I will be releasing a D6 version in the near future.

#18

davidwhthomas - September 8, 2008 - 08:30
Status:needs review» closed

Drupal 6 development version has been released and will appear for download once the packaging script has run

#19

ArjanLikesDrupal - September 19, 2008 - 00:13

Thanks!

 
 

Drupal is a registered trademark of Dries Buytaert.