GREAT WORK ! :)

one thing that would make it even nicer is to integrate with taxonomy image and allow users to add images when changing the description of a particular term.

As is now I use taxman... but then go to the tax images area to deal with images ... doable... just a pain .. . :)

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

vph’s picture

agreed

tacituseu’s picture

Version: 5.x-1.x-dev » 6.x-2.x-dev
Status: Active » Needs review
FileSize
19.25 KB
2.82 KB
tacituseu’s picture

Added missing Drupal.attachBehaviors().

Majdi’s picture

subscribing

Andrew Gorokhovets’s picture

subscribing

schiavone’s picture

This patched failed and had to apply manually. At first it seemed that I was unable to upload an image with no error. Tried in both Mozilla and Chrome.

As it turns out Taxonomy_Image has trouble with spaces in the filename.

Thanks.

SeanA’s picture

First, thanks for contributing the patch. (And thanks also to the maintainers of the Taxonomy Manager module. It's just what I was looking for.) Patch in #3 applies successfully with minor offsets to latest dev release. However, it's not quite working 100%, there are a few issues.

- Coding style: control structures should conform to Drupal coding standards to improve readability. For example...

if( module_exists('taxonomy_image') ) {
      foreach( array('taxonomy_image_current_image_delete', 'taxonomy_image_external') as $key ) {
        if( isset($param[$key]) ) $term[$key] = $param[$key];
      }
    }

should be...

if (module_exists('taxonomy_image')) {
      foreach (array('taxonomy_image_current_image_delete', 'taxonomy_image_external') as $key) {
        if (isset($param[$key])) $term[$key] = $param[$key];
      }
    }

- When adding a new term using the taxonomy manager, if I enter a term description and attach an image at the same time, the description field is not saved. If I do these operations separately, the fields are saved correctly.

- On the edit term overlay, the term image fieldsets have acquired "move" handles like the term title, for moving the overlay around. It's sort of confusing to have these extra handles in the middle of the form. EDIT - This is when using the double tree layout.

--
Taxonomy Image

SeanA’s picture

Title: Working with taxonomy image » Integration with Taxonomy Image module
Status: Needs review » Needs work

Not trying to contradict schiavone... I should have mentioned #3 applies successfully using -p1.

ivnish’s picture

Issue summary: View changes
Status: Needs work » Closed (outdated)