Anonymous users can see the "products" tab on nodes

Not sure why, but there is an access argument set to a function, I just changed the function till I find out why they did it this way:

/**
 * Chekcks if the node is a products display node and that user has access to it.
 */
function commerce_pdm_access_products_tab($node) {
  // return user_access('access content') && _commerce_pdm_get_node_reference_field_name($node) !== FALSE;
  return user_access('access product display manager');
}

+1

I've gone with:

/**
 * Chekcks if the node is a products display node and that user has access to it.
 */
function commerce_pdm_access_products_tab($node) {
  return user_access('access product display manager') && _commerce_pdm_get_node_reference_field_name($node) !== FALSE;
}

and this works fine for me.

+1 Can't see a reason why we need to confuse the users visiting the site with a clearly Administrative function.

subscribing

Subscribing

subscribing

Here is a patch for this issue:

- corrected permission in commerce_pdm_access_products_tab from "access content" to "access product display manager"
- typo in the comments for the above function
- corrected capitalisation of "Access product display manager" in hook_menu

Patch works fine for me - thanks!

#9 is a winner! Lets get this committed.

#9 fixed the issue for me as well.

vote +1 on #9!

Patch on #9 worked for me as well.

Thanks superstar!

Hi,
I do not see commits from july 20. Is this module still maintained please?
I am looking for a module like this as add on to Commerce_BPC.
Thanks a lot in advance for your reply.
greetings, Martijn

+1 on #9

Yeap, #9 worked for me as well.

Superstar's patch in #9 has worked for me too, thanks.

Tnx superstar , patch #9 worked.
Like@summit I would like to know if this module is still maintained please?

^ yeah that. It seems this project has been abandoned. It's a total shame considering how vital it is to commerce.

Hi guys, I'm looking for maintainers of this module just because I don't have any time for it myself ATM. If there is anyone that would like to become a co-maintainer of the module, please let me know.

I really would like to help you, but I'm not a developer :(

Thanks! Committed this.

I have the latest version of core and all modules and im still getting this issue.

Could someone provide some instructions for the solution from #4? If I remove the php tags and add it to my template.php file i get a white screen for my site. thanks

Hi jdln the patch is applied to the module file not template. See the instructions here for how to apply patches manually. http://drupal.org/node/534548 Easy once they've been explained. Yes, this patch worked for me. I do hope this module gets maintained again.

There is a module named Commerce Auto Product Display which does similar thing like this module. You might want to use it because this module is no longer maintained.

Hi,
I think now the http://drupal.org/project/inline_entity_form is the way to move forward!
Greetings, Martijn

Does anyone actually know how to fix this issue which only came about after we updated the core from drupal 7.14 to 7.18.

This has been fixed in the 7.x-1.x-dev (development) version. It's waiting to be pushed with a formal stable release. If you really need this fix right now: apply the code in the patch in #9 to your stable module installation.

Closing this for now again.

On a related note: I'm going to take a bit of time to run through the issue queue and revisit bug reports.

Marking as fixed.