Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Are there any plans for porting to D8?
Comment | File | Size | Author |
---|---|---|---|
#7 | 2754715-autocomplete-deluxe-d8-port-7.patch | 52.08 KB | edwardchiapet |
#7 | interdiff-2754715-4-7.txt | 32.64 KB | edwardchiapet |
#4 | 2754715-autocomplete_deluxe-d8-port-4.patch | 88.97 KB | id.tarzanych |
Comments
Comment #2
DYdave CreditAttribution: DYdave at DAVYIN Internet Solutions / 戴文信息科技有限公司 commentedStandardized title.
Comment #3
id.tarzanych CreditAttribution: id.tarzanych at Internetdevels commentedPlease review my patch
It's pretty raw and needs work on customization improvement and code clean-up, but I believe it can be a start point of 8.x-2.x-dev branch.
Comment #4
id.tarzanych CreditAttribution: id.tarzanych at Internetdevels commentedFixed field title display
Comment #5
willyk CreditAttribution: willyk commentedid.tarzanych did you see this issue? https://www.drupal.org/node/2838576#comment-11944808
Comment #6
m.abdulqader CreditAttribution: m.abdulqader at Sprintive commented+1
Comment #7
edwardchiapetI was able to work off of @id.tarzanych's patch and make some updates based on that - including fixing PHPCS warnings/errors, adding ability for reordering, and a setting for allowing auto-creation of new terms.
I've also created a sandbox project with the D8 port for testing - https://www.drupal.org/sandbox/edwardchan/2888186.
Comment #8
JonMcL CreditAttribution: JonMcL commentedGreat work @id.tarzanych and @edwardchiapet!
@edwardchiapet: I did run into problems with the allow auto-creation of new terms functionality. I think you need to modify the
getAutocreateBundle
function to be like:This is a direct copy from the
EntityReferenceAutocompleteWidget
class.Also, is the
new_terms
widget setting even necessary? Did you consider re-using theauto_create
field setting instead?Comment #9
LNakamura CreditAttribution: LNakamura at Mediacurrent commentedThanks, @JonMcL! That does indeed fix the auto-creation of new terms.
As for the utility of
new_terms
, I'd say that it allows for flexibility on a per-field basis: one field could allow new terms, while another does not, though both would be gated byauto_create
.Comment #10
LNakamura CreditAttribution: LNakamura at Mediacurrent commentedHi, all - I incorporated @JonMcL's fix, cleaned up a few typos and a coding standard issue, and created 8.x-1.x, -dev, and -beta1 releases.
We'll be working on porting a few recent 7.x-2.x-dev fixes in the near future.
Thanks!
Comment #11
LNakamura CreditAttribution: LNakamura at Mediacurrent commentedComment #12
LNakamura CreditAttribution: LNakamura at Mediacurrent commentedClosing this issue since the basic port is done - new issues should be created for bugs, etc., against the D8 code.
Comment #13
LNakamura CreditAttribution: LNakamura at Mediacurrent commentedEdit to credit @JonMcL on the issue....