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

CommentFileSizeAuthor
#9 media_gallery-remove_taxonomy_shenanigans-1686498-9.patch4.8 KBjsagotsky
#8 media_gallery-remove_taxonomy_shenanigans-1686498-8.patch5.24 KBjsagotsky
#3 media_gallery-remove_taxonomy_shenanigans-1686498-3.patch11.41 KBjsagotsky

Comments

c0ldfury’s picture

c0ldfury commented

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

selinav commented

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

jsagotsky’s picture

jsagotsky commented
StatusFileSize
new media_gallery-remove_taxonomy_shenanigans-1686498-3.patch11.41 KB

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

lsolesen commented
Status: Active » Closed (duplicate)

Closing as a duplicate of #1333674: Conflicts with other modules so media_gallery_collection not called which creates Huge Images On Main Gallery Page

selinav’s picture

selinav commented

#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

jsagotsky commented

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

selinav’s picture

selinav commented

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

jsagotsky commented
StatusFileSize
new media_gallery-remove_taxonomy_shenanigans-1686498-8.patch5.24 KB

Rerolled against current version.

jsagotsky’s picture

jsagotsky commented
StatusFileSize
new media_gallery-remove_taxonomy_shenanigans-1686498-9.patch4.8 KB

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

caspercash’s picture

caspercash commented

How about the #9 patch? Is it working?