CommentFileSizeAuthor
#9 avoid-duplicates-references.patch804 bytesiyyappan.govind
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

gokulnk created an issue. See original summary.

gokulnk’s picture

gokulnk CreditAttribution: gokulnk at Azri Solutions for CyberSafe commented

This pull request tries to fix that by removing the duplicates before updating the input field.

https://github.com/drupal-media/entity_browser/pull/92

slashrsm’s picture

slashrsm CreditAttribution: slashrsm at Examiner.com commented
Status: Needs review » Fixed

Merged. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

Bunty Badgujar’s picture

Bunty Badgujar CreditAttribution: Bunty Badgujar commented

Hi,

I am facing this issue in entity_browser 8.x-1.0-rc1 with Drupal core 8.2.6. The committed patch is missing from js/entity_browser.common.js

Thanks

forX’s picture

forX CreditAttribution: forX commented

I have the same problem than #bunty_badgujar. (duplicate selected items with Version: 8.x-1.0-rc1)

iyyappan.govind’s picture

iyyappan.govind
he/him
Primary language Tamil
Location Chennai, India
CreditAttribution: iyyappan.govind at UniMity Solutions Pvt Limited commented

Same issue in 8.x-2.2

iyyappan.govind’s picture

iyyappan.govind
he/him
Primary language Tamil
Location Chennai, India
CreditAttribution: iyyappan.govind at UniMity Solutions Pvt Limited commented
Version: 8.x-1.x-dev » 8.x-2.x-dev
Issue summary: View changes

I am using Entity Browser version is 8.x-2.2. Facing same issue.

iyyappan.govind’s picture

iyyappan.govind
he/him
Primary language Tamil
Location Chennai, India
CreditAttribution: iyyappan.govind at UniMity Solutions Pvt Limited commented
FileSize
avoid-duplicates-references.patch804 bytes

Here is the patch for 8.x-2.x. Please review this patch.

Thanks

amykhailova’s picture

amykhailova CreditAttribution: amykhailova at Kalamuna commented

I tested the patch it works for the latest version of the module. Please reopen and commit the patch.

loze’s picture

loze CreditAttribution: loze commented

Yes, works for me too. I agree this should be added.

gwvoigt’s picture

gwvoigt
Location London, ON 🇨🇦
CreditAttribution: gwvoigt at Northern Commerce commented

This patch works for me on Firefox 86.0 (64-bit), but fails on Chrome Version 88.0.4324.192 (Official Build) (x86_64). Is it a duplicate of https://www.drupal.org/project/entity_browser/issues/3074457 ?

CRZDEV’s picture

CRZDEV CreditAttribution: CRZDEV at Metadrop commented

Those with browser related issues after applying patch, please take a look at https://www.drupal.org/project/entity_browser/issues/3074457#comment-140...