Closed (fixed)
Project:
Feeds Tamper Importer
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
21 Feb 2018 at 12:43 UTC
Updated:
27 Feb 2018 at 05:57 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
firfin commentedI just discovered something else, when visiting /admin/structure/feeds/[my_feeds_importers]/tamper/import I am getting an access denied error. I double checked but the "Administer Feeds Tamper Importer" IS set for my role.
Comment #3
firfin commentedOk, I found the problem. In hook_permission you declare the permission
'administer feeds_tamper_importer' => array(
But in hook_menu you use:
'access arguments' => array('administer feeds_tamper_import'),
That last one has 'er' missing at the end.
Patch coming soon
Comment #4
iyyappan.govindHi, Thanks for the issue. Actually this issue is fixed in dev version. Thank you
Comment #5
firfin commentedOk, the dev version already had the proposed fix in it. So instead, just a patch with more usage of importer instead of import to prevent these kind of mistakes in the future. Also added some documentation.
Bit annoying there is only one 'stable', but not functional version of the module. Might it be possible to do a new release please?
Comment #7
iyyappan.govindThanks for the patch, Committed.. Will release the new version soon. Thank you
Comment #8
firfin commentedThank you and good luck with creating a new version. If you need any help, let me know please.
Comment #9
iyyappan.govind