While I was debugging some issues with the translation of commerce_product with new entity_translation dev version, I added a dsm($op) in commerce_product_access(). Then when I visited a product edit page, I found that following printed on product edit page.


    update

    update

    update

    update

    update

    create

    create

    update

This let me to the conclusion that we are checking access for $op ='update' 6 times and $op='create' 2 times. I am not sure if this is needed. Also why should we add access check to 'create' op in the product edit screen?

If this is an intended behaviour, can anyone pls take sometime to explain why? It could be useful for the community to know the logic behind this.

Thanks

Comments

rszrama’s picture

Category: bug » task

Can we confirm that there aren't other modules interacting with the access checks?

jsacksick’s picture

Issue summary: View changes
Status: Active » Closed (won't fix)

Closed as Drupal 7 is well beyond EOL.