At the moment crumbs does not appear anywhere on the configuration page. I'm going to upload a patch shortly which fixes that...

UPDATE:
This is supposed to be fixed, but you need to:
- drush cc menu (or just drush cc all), that is. Or from the web ui, just clear the caches.
- go to the permissions page, and enable "Administer Crumbs". (if you don't already have an admin role, where this should be enabled automatically)
- the menu item should appear at Structure > Crumbs > Settings.

If this all doesn't help, post here again.

Comments

bforchhammer’s picture

StatusFileSize
new1.56 KB

Here we go...

Attached patch also fixes the debug form (was submitting to wrong path), and removes a E_NOTICE message.

bforchhammer’s picture

StatusFileSize
new1.9 KB

Well, that was too hasty. Attached is a patch that's actually working; now crumbs shows up in the user interface section on the configuration page.

donquixote’s picture

Not sure.
On the few D7 sites that I work on, the only thing I found in "User interface" was "Shortcuts". I don't see that Crumbs belongs in there.

bforchhammer’s picture

Well, tbh I didn't really know where to put it. I have "Delta blocks" and "Menu block" in the "User Interface", and with breadcrumbs being an UI element, that actually seemed to fit nicely. I don't really care either way, just though that it should appear somewhere so people don't have to know and type in the path...

donquixote’s picture

Hm.
I'm going to put it into "Structure" for now. This does not make much sense either, but there it is close to "Menu". Also it is close to "Context", which at some day it could

For some reason I don't like the term "User interface". It is so generic, it could be anything.

Anyway, thanks for caring!
The next release will have this improvement, or something equivalent. Until then, you can use your own modified version.

-------

Btw, I am always happy about feedback!
Esp,
How well does this work compared to other breadcrumb-customizing modules?
Is there any use case for a breadcrumb that can not be achieved with Crumbs?
How many % of site builders understand the configuration page, with those hierarchic weights?
How easy/difficult and practical is the API ?

(can be in another issue, if you like)

bforchhammer’s picture

joelstein’s picture

Regardless of where you want to put it, it won't show up for anyone except user #1, because you have hook_perm(), which is D6, and you need to use hook_permission():

/**
 * Implements hook_permission().
 */
function crumbs_permission() {
  return array(
    'administer crumbs' => array(
      'title' => t('Administer Crumbs'),
    ),
  );
}
donquixote’s picture

Woot!
It's great having someone other than me look at this stuff.
Big thanks!

joelstein’s picture

You're welcome. Are you going to commit it?

donquixote’s picture

On the way.
http://drupalcode.org/project/crumbs.git/commit/716b9a424997638460f2a6e4...

Going to be released as 7.x-1.0-alpha4.

joelstein’s picture

Fantastic. Thanks!

Sherbet’s picture

is it me or does Crumbs still not appear to have a config page? I'm using 7.x-1.0-alpha4

donquixote’s picture

Hi,
you need to
- drush cc menu (or just drush cc all), that is. Or from the web ui, just clear the caches.
- go to the permissions page, and enable "Administer Crumbs". (if you don't already have an admin role, where this should be enabled automatically)
- the menu item should appear at Structure > Crumbs > Settings.

I have this working on two sites now, and it does ok.

Sherbet’s picture

That seemed to fix it!

However, upon moving lines around in the textarea I get the following error:
Notice: Undefined offset: 1 in crumbs_admin_form_submit() (line 208 of C:\xampp\htdocs\final\sites\all\modules\crumbs\crumbs.admin.inc).

donquixote’s picture

In the crumbs.admin.inc, if you replace the respective code like this, does it help?

  foreach ($lines as $line) {
    $line = trim($line);
    list($key, $title) = explode(' ', $line, 2) + array(NULL);
donquixote’s picture

Status: Active » Fixed

#15 committed and release as 7.x-1.0-alpha5

Sherbet’s picture

Updating to the latest build did not fix the issue, unfortunately.

donquixote’s picture

I updated the issue summary. Can you check?

Sherbet’s picture

I was actually referring to the issue with the offset which was supposedly fixed in alpha5, but does not seem to work for me. The initial issue of the menu not showing up was solved by clearing caches.

donquixote’s picture

Oops. Ok. Can you do a bit of debugging yourself?

Status: Fixed » Closed (fixed)

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

donquixote’s picture

Just pushed another fix for the bug reported in #14.
It had to be + array(NULL, NULL) instead of just + array(NULL).
This is going to be 7.x-1.0-alpha6.

donquixote’s picture

Issue summary: View changes

Steps to check, before you complain.