Why does it need to do this? A media gallery isn't going to be the only thing people need taxonomy lists for.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

c0ldfury’s picture

ok, so other modules deal with this by doing something like:

function media_gallery_menu_alter(&$items) {
  // If ctool's page manager is active for the path skip this modules override.
  // Also views module takes over this page so this won't work if views enabled.
  if (variable_get('page_manager_term_view_disabled', TRUE))  {
	  $items['taxonomy/term/%taxonomy_term']['page callback'] = 'media_gallery_list_galleries';
	  $items['taxonomy/term/%taxonomy_term']['file'] = 'media_gallery.pages.inc';
	  $items['taxonomy/term/%taxonomy_term']['module'] = 'media_gallery';
  }

But that isn't acceptable in this case, because it ruins the gallery list.

selinav’s picture

I have the same problem when I override taxonomy display in views.

jsagotsky’s picture

I'm not a fan either. I made a patch that comments out the taxonomy shenanigans. It removes the menu_alter that hijacks /taxonomy and prevents hook_install from creating taxonomy and terms. I don't expect this to be applied to master, but it might help someone who wants to use media_gallery without the taxonomy business.

lsolesen’s picture

selinav’s picture

#3 thanks for the patch.
I've seen that you replace the type media of media_gallery_media field by a type file. Is it still compatible with plupload?

jsagotsky’s picture

selinav, not sure. I haven't tried plupload. Let me know what you find.

selinav’s picture

It doesn't work with plupload. If I uninstall plupload, I can't add image, the media selector is displayed but when I save, the modal box close and doesn't save my images.

I have also a problem with colorbox which is not resize (not cause by your patch). I will uninstall this module and create my content type gallery with a media field and make a view to see all galleries.

It's a shame, I liked the drag and drop, the plupload integration and the design.

jsagotsky’s picture

Rerolled against current version.

jsagotsky’s picture

That last one didn't even work... rerolling.

caspercash’s picture

How about the #9 patch? Is it working?