Enabling the default image support needs to happen through a function that other modules and installation profiles can use. Right now, UberDrupal redirects to admin/store during the installation process, because default image support is enabled through a page callback that assumes you want the redirect. Instead, there should be a separate page callback that calls the API function to enable image support. The API function itself should be in uc_product.module.

CommentFileSizeAuthor
#1 imagecache_api.patch5.36 KBIsland Usurper

Comments

Island Usurper’s picture

Island Usurper commented
Assigned: Unassigned » Island Usurper
Status: Active » Needs review
StatusFileSize
new imagecache_api.patch5.36 KB

The code in that page callback looks robust enough to be used in uc_product_enable() as well, because it shouldn't create duplicate presets for the catalog module. The new function's name is uc_product_add_imagecache_presets().

rszrama’s picture

rszrama commented
Status: Needs review » Fixed

Reviewed... looks good. Added a small notification message and committed.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.