I have added a new entity claim along with fields. I went to test it as a different user on one of the nodes and everything appeared to work but it is not showing up on Content > Claims admin/content/claims. I went back to try and claim it again but now it says You have already claimed for ...
Am I missing something?
| Comment | File | Size | Author |
|---|---|---|---|
| #12 | pending-claims-admin-fix-2666662-12.patch | 1.26 KB | monstrfolk |
| #11 | Capture1.PNG | 19.98 KB | monstrfolk |
| #11 | Capture.PNG | 17.45 KB | monstrfolk |
Comments
Comment #2
nikunjkotechaHi freshpage, can you add more information for this? may be screenshot or exported feature of your configuration to try and check? Flow you mentioned seems to be perfectly fine for me.
Comment #3
monstrfolk commentedsame problem.
Comment #4
monstrfolk commentedComment #5
monstrfolk commentedComment #6
monstrfolk commentedThe problem is the $arg variable in the menu callback.
In entity_claim.module file...
menu callback for the default claim admin page (pending claims) - no page argument set.
In entity_claim_content.inc file....
default $type is pending but the system (drupal) is setting 'node_admin_content' as the page argument by default.
So $type can never be set to pending because the entity_claim_content function is called with 'node_admin_content'.
Patch fixes this problem.
Comment #7
monstrfolk commentedComment #8
miteshmap@monstrfolk: I didn't understand the issue here. We defined a menu item here.
$items['admin/content/claims']Given a specific page callback for this menu item
'page callback' => 'entity_claim_content',Right?If I do not pass the page argument, and default value is not pass to the page callback function, That will give me notice of undefined argument.
So, How could this menu item can call
node_admin_contentas default page argument?Comment #9
monstrfolk commentednode_admin_contentis being passed toentity_claim_contentnot fromentity_claim_menufunction, but from some other function. I don't know what, but I do know that by settingpage arguments' => array('pending'),inentity_claim_menuenforces thatentity_claim_contentwill never be passed an unwanted value overriding the desired default value.Comment #10
monstrfolk commented@miteshmap... were you able to look at this issue?
Comment #11
monstrfolk commentedI have reverted this patch. Attached are two pictures. One is a dpm in the code and the other is the output of dpm.
Comment #12
monstrfolk commentedMy last patch did not assign 'pending' when viewing admin/content/pending page. New patch corrects this. This works and solves the issue. I have double and tripled checked this solution and it works.
Comment #13
monstrfolk commentedComment #14
nikunjkotechaComment #16
nikunjkotechaFixed in dev.
Comment #17
monstrfolk commented