WYSIWYG + IMCE integration [#1328952]

when you uploading image with the same name, like image in imce, it's path will be altered by path, specified in filefield paths
and this is the bug, because if for image uploaded into filefield it will works fine - path will be altered like "files/.." to "files/images/.." and file will be moved.
But in the case of textfield with WYSIWYG (+imce, ckfinder probably too) path will be changed from "files/imce/.." to "files/imce/images/.."

patch attached

Comment	File	Size	Author
	filefield_paths-6.x-1.x-dev-wysiwyg-imce-integration.patch	922 bytes	chingis

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

chingis CreditAttribution: chingis commented 2 November 2011 at 04:33

Category:

task

» bug

Comment #2

Deciphered CreditAttribution: Deciphered commented 2 November 2011 at 19:20

Status:

Active

» Needs work

+++ b/sites/all/modules/contrib/filefield_paths/filefield_paths.module
@@ -572,6 +572,12 @@ function _filefield_paths_text_fields($node) {
+            if (isset($node->{$name}[$key]['format']) && ¶
+                ($profile = wysiwyg_get_profile($node->{$name}[$key]['format'])) && ¶
+                isset($profile->settings['buttons']['imce'])) {

Formatting of this conditional statement doesn't comply with the Drupal coding standards, I'd recommend assigning the $profile value outside of the condition to keep it cleaner.

+++ b/sites/all/modules/contrib/filefield_paths/filefield_paths.module
@@ -572,6 +572,12 @@ function _filefield_paths_text_fields($node) {
+                ($profile = wysiwyg_get_profile($node->{$name}[$key]['format'])) && ¶

wysiwyg_get_profile() will cause critical error if Wysiwyg module is not present.

Chees,
Deciphered