Block for link to entity claim. Options included in block are...

  1. Link text
  2. Link target
  3. Link extra classes
CommentFileSizeAuthor
#15 entity-block-2642996-15.patch4.57 KBmonstrfolk
#14 entity-block-2642996-14.patch4.62 KBmonstrfolk
#13 entity-block-2642996-13.patch4.61 KBmonstrfolk
#10 entity-block-2642996-9.patch6.05 KBmonstrfolk
#4 entity-block-2642996-4.patch4.68 KBmonstrfolk
#3 entity-block-2642996-3.patch4.45 KBmonstrfolk
entity-block-2642996-1.patch4.6 KBmonstrfolk

Comments

monstrfolk created an issue. See original summary.

monstrfolk’s picture

monstrfolk commented
Issue summary: View changes
monstrfolk’s picture

monstrfolk commented
StatusFileSize
new entity-block-2642996-3.patch4.45 KB

Cleaned up output slightly.

monstrfolk’s picture

monstrfolk commented
StatusFileSize
new entity-block-2642996-4.patch4.68 KB

Fixed warning.
Notice: Undefined variable: target in entity_claim_block_view()

monstrfolk’s picture

monstrfolk commented
Version: 7.x-1.x-dev » 7.x-1.0-beta3
miteshmap’s picture

miteshmap
Primary language Gujarati
commented
Status: Reviewed & tested by the community » Needs work

I found two issues with this,

  1. Patch didn't apply with git
  2. The link text is static -`Claim Entity`, end user wouldn't like to see this. It should be dynamic, admin would like to choose the text to display.

Third thing, With the dynamic text, I would not choose the same text for different entities (IF I'm allowing multiple entity claim). I would need more than 1 block to place at different locations for different entities OR with one block I would prefer to configure text for each entities claim link.

monstrfolk’s picture

monstrfolk commented

Thanks. I will fix these issues and resubmit.

monstrfolk’s picture

monstrfolk commented

@miteshmap...

2. Claim Entity text is dynamic. 

$form['entity_claim_link_text'] = array(
        '#type' => 'textfield',
        '#title' => t('Link text'),
        '#default_value' => entity_claim_link_text(),
      );

Making a block per each entity able to claim. I will resubmit in changes.

monstrfolk’s picture

monstrfolk commented
Issue summary: View changes
monstrfolk’s picture

monstrfolk commented
StatusFileSize
new entity-block-2642996-9.patch6.05 KB

One block per entity claim bundle added. Cloned git tonight and created this patch. It should apply. Also added an option to redirect to previous page after claim submit. It would be nice to combine this option with the option to redirect to a custom page like on the claim settings page.

monstrfolk’s picture

monstrfolk commented
Status: Needs work » Needs review
monstrfolk’s picture

monstrfolk commented
Issue summary: View changes
monstrfolk’s picture

monstrfolk commented
StatusFileSize
new entity-block-2642996-13.patch4.61 KB

Removed...

  • Redirect to previous page after submitting claim form
  • Link to user login for anonymous users

Blocks follow behavior of Entity Claim instance settings.

monstrfolk’s picture

monstrfolk commented
StatusFileSize
new entity-block-2642996-14.patch4.62 KB

Fix on token redirect. Should be good now.

monstrfolk’s picture

monstrfolk commented
StatusFileSize
new entity-block-2642996-15.patch4.57 KB

added check for entity_claim_add_access

nikunjkotecha’s picture

nikunjkotecha
he/him
Primary language Gujarati
Location India, Gujarat, Rajkot
commented
Status: Needs review » Closed (fixed)