wbrowar / adminbar Goto Github PK

Hi, I updated to the latest version of your plugin and when accessing a page with "category" not "entry" I get the following exception thrown:

Craft\CategoryModel and its behaviors do not have a method or closure named "section".

In ideas please?

Thanks in advance.
Daniel

When I try to add the adminbar to the site using either method I get an error stating "Impossible to invoke a method ("can") on a NULL variable ("")".

The docs use the Twig output tag:

{{ hook 'renderAdminBar' }}

The correct tag is the action tag:

{% hook 'renderAdminBar' %}

I've run into couple of issues enabling this on a server running php 5.3.x

It worked without a hitch locally, but on the production server I'm getting:

Fatal error: Using $this when not in object context in ../craft/plugins/adminbar/AdminbarPlugin.php on line 18

I also had to change https://github.com/wbrowar/craft-admin-bar/blob/master/adminbar/services/AdminbarService.php#L23-L30 to use array() rather than just []

Any help would be appreciated and looking forward to your multi entry solution ;-)

Hi,

I think its self explanatory. When clicking the edit link, it should link to the current locale.

Thanks,
Eric

Love this, as I do a lot of Drupal work and find that content authors love having an admin bar at the top of the screen for editing the current content.

That being said, I cannot use this as it currently stands as I often override the cpTrigger, so /admin/settings does not work for me.

If instead you grabbed the cpTrigger (below) in your variables, it would make the settings link more versatile when users have overridden their default trigger:

craft()->config->get('cpTrigger') . '/settings'

If the repo is restructured so that the /services ,/templates, /variables and AdminBarPlugin.php are at the top level of the repo, then others could use it as a Git Submodule in their Craft projects under the /plugins dir (this make updates, pr's etc, easier)

LICENSE and README, etc can stay at the top, as Craft will just ignore those.

Though, technically, I could wrap the admin bar in a {% if not craft.request.isLivePreview %} it would be better if the admin bar did not show up in live preview by default.

I've installed Adminbar and it shows up and works just fine on the front end. The problem is, when Adminbar is enabled, none of the bottom row of maintenance stuff on the Settings screen works (Asset indexes, clear cache, backup db, etc). When i disable or uninstall the plugin, they all work just fine.

Would prefer the "Edit" button in the adminbar to just say "Edit" without appending the section name. Our section names don't always make sense and it makes this button a little ugly/confusing/inconsistent.

Some examples
Edit Pages
Edit For Rent
Edit Inventory

To recreate:

• Add a custom link on the on the settings page: http://share.barrelstrength.co/3O0Z0O100029
• Use the delete button on the right to delete it.
• Save your settings.
• Reload your settings and the link is still there.