Closed (outdated)
Project:
Taxonomy Manager
Version:
6.x-2.x-dev
Component:
User interface
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
28 Sep 2008 at 01:33 UTC
Updated:
13 Feb 2020 at 09:17 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
vph commentedagreed
Comment #2
tacituseu commentedSee #846598: inside filefield_js() use drupal_json instead of drupal_to_js. Headers are of the wrong content type. and https://github.com/malsup/form/commit/b3e48921a64afe8d3b2ae11b10aa24f9ab... for some background behind adding
print drupal_to_js().Comment #3
tacituseu commentedAdded missing
Drupal.attachBehaviors().Comment #4
majdi commentedsubscribing
Comment #5
Andrew Gorokhovets commentedsubscribing
Comment #6
schiavone commentedThis 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.
Comment #7
SeanA commentedFirst, 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...
should be...
- 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
Comment #8
SeanA commentedNot trying to contradict schiavone... I should have mentioned #3 applies successfully using -p1.
Comment #9
ivnish