wrong url in "File attachments"

Hello,
I need this too.

Thanks,
David

Please consider to fix it. Although the function still works, but users will be confused.

This is a known issue. However, I really don't know how to fix it given the way the upload module works. :-/ If anyone has suggestions on how to do so, please share. (Or a patch would be even better.)

A stupid way: modify modules/upload/upload.module, remark the 2 lines 777,778 and user will NOT see the url. If they doesn't see it, then they will not be confused :p

      //$description = file_create_url((strpos($file->fid, 'upload') === FALSE ? $file->filepath : file_create_filename($file->filename, file_create_path())));
      //$description = "<small>". check_plain($description) ."</small>";

But I think if we need this function, maybe we need to modify upload module, not upload path module.

I tried to modify upload.module, it seems ok.

old:

      //$description = file_create_url((strpos($file->fid, 'upload') === FALSE ? $file->filepath : file_create_filename($file->filename, file_create_path())));

new:

      $description = file_create_url((strpos($file->fid, 'upload') === FALSE ? $file->filepath : file_create_filename($file->filename, token_replace(variable_get('uploadpath_prefix', ''), 'node', $node))));

But I found another issue, I can use "u[author-uid]" as the prefix in upload path module, but I cannot use it in upload module. Is it due to the node is not generated yet, so the [author-uid] is null?
When I change to use "u[user-id]" as the prefix in upload path, it's ok in both upload and upload path.

Thanks dennys, although modifying core is not something this module can do. :-) As for the other issue, please file that as a new issue. It's very hard to track issues that are mixed into one thread. Thanks.

Subscribe. Will be good to see the real path in state after uploading file, but I don't know how to do it without hacking the upload.module. Maybe use a hook_form_alter() for upload file submitting form?

Subscribing.

I changed upload.module as well to solve this, using a str_replace to replace "files/" with "files/".$usersrole where $usersrole is the logged in user's role. This custom code was on the line after the one stated above, where I use the str_replace on the $description=" line. I have the uploadpath module customized to upload files to user's ROLES. Using a customized token module with uploadpath and changing the upload.module file, I have a nice setup that works for Drupal 5.7.

But you hacked core...

Is this feature of correct filepath at upload (and before the node post) really impossible to somehow get implemented in Uploadpath module?

I just installed Uploadpath module and before that moved all the old files to new locations, changed all the paths in files table, corrected all the urls in node_revisions table and it all works perfectly, besides this feature - my problem is, that I have a lot of users on my site and majority will have problems with inserting images. Yes, I'll make some banner on the content/add page, but this feature would be far better solution. If there is any way to help.

At least someone gives me the change of Drupal 6.x upload module (even though I wouldn't like to change core).

One solution would be to have the module check the Body field for the old paths and replace them with the new paths after relocating/renaming the files.

While it would be great to know the correct path on upload, node tokens are not set in stone until the node has been saved.

I started working on an uploadpath_form_alter hook to avoid hacking core. It's possible to get working for simple cases.

The major problem with finding a gereral solution is that lots of tokens will not work when adding a new node (node object/information is missing). Possibly some node information could be gathered in a temporary node object, but it world not be 100% correct.

Also, it would be good for implementing this to extract the rewriting code of uploadpath_nodeapi into a separate function.

Here's the code if anyone wants to have a try (working with only a standard prefix):

function uploadpath_form_alter($form_id, &$form) {
  if ($form_id == 'upload_js') {
    if (isset($_POST['vid']) && is_numeric($_POST['vid'])) {
      //Load the node object
      $node = node_load(array('vid' => $_POST['vid']));
    }
    else {
      // This is a new node.
      $node = new stdClass();
    } 
    // Alter the description of all newly uploaded files
    for ($i=0; isset($form['files']['upload_'.$i]); $i++) {
      $description = file_create_url(file_create_filename($form['files']['upload_'.$i]['filename']['#value'], token_replace(variable_get('uploadpath_prefix', ''), 'node', $node)));
      $form['files']['upload_'.$i]['description']['#description'] = $description;
    }
  }
}

Thanks for the idea, I made some tweaking with bjorpe's code and now it really works for default token [type/yyyy/mm] and any other token which contains only type and/or date items. Someone may proceed with tweaking and at least add other type-based tokens support (now it is possible).

function uploadpath_form_alter(&$form, $form_state, $form_id) {
  if ($form_id == 'upload_js') {
  	if (isset($_POST['vid']) && is_numeric($_POST['vid'])) {
      //Load the node object
      $node = node_load(array('vid' => $_POST['vid']));
    }
    else {
      // This is a new node.
      $node = new stdClass();
      $node->type=str_replace('_node_form', '', $form["#post"]["form_id"]);
      $node->created=time();
    }
  	// Alter the description of all newly uploaded files
    foreach ($form['files'] as $k=>$v) {
      if (is_numeric($k)){
        $description = file_create_url(file_create_filename($form['files'][$k]['filename']['#value'], token_replace(variable_get('uploadpath_prefix', '[type]/[yyyy]/[mm]'), 'node', $node)));
        $form['files'][$k]['description']['#description'] = $description;
      }
    }
  }
}

How about some client-side code (jquery?) that somehow gets triggered after the page render is finalized and then dynamically modifies the link in the DOM?

Hmm, subscribing

When uploading a file using standard file attachment feature with AHAH callback it uploads the file in file/ path.
If we save the node or content the upload path module uploads the file in the folder path we configured in the admin settings.
This should not be happened. Any solutions?