Somewhere along the line uc_catalog_url was removed
function uc_catalog_update_2() {
variable_del('uc_catalog_url');
return array(array('success' => TRUE, 'query' => "variable_del('uc_catalog_url')"));
}

But it still has remnants and personally I would have preferred to have this variable as I am having breadcrumb issues since we made a custom catalog page.

Anyways the line is

$crumbs[] = l(variable_get('uc_catalog_name', t('Catalog')), variable_get('uc_catalog_url','catalog'));

Line 171 uc_catalog.module.

Comments

cha0s’s picture

Status: Active » Needs review
StatusFileSize
new706 bytes

Might as well, unless someone's got another idea!

kylebrowning’s picture

The patch is great, but what about setting the url, what if i did have a custom URL for my Catalog?

cha0s’s picture

Is it possible to use a URL alias?

cha0s’s picture

StatusFileSize
new2.17 KB

Okay, I went over this issue again and got another patch now. It fixes other problems with the catalog module.

Front page wasn't resolved correctly.
Unused URL variable being invoked.
Try to create a URL alias before throwing away the user setting.

itsmahitha’s picture

Reviewed and Tested.

tr’s picture

Version: 5.x-1.7 » 6.x-2.x-dev
longwave’s picture

Status: Needs review » Fixed

Ported to 6.x-2.x and committed.

Status: Fixed » Closed (fixed)

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