Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

N°8245 - External key not saved in n-n link in edition#703

Conversation

bdalsass
Copy link
Contributor

@seehttps://support.combodo.com/pages/UI.php?operation=details&class=Bug&id=8245

Bug introduced with bug N°8245 since N°7264 Update selectize.js from 0.12.4 to 0.15.2
This version of Selectize library reset the input change listeners on initialization.

Needs a better implementation because the code is quite ugly.

@bdalsassbdalsass added bugSomething isn't working internalWork made by Combodo labelsMar 18, 2025
@bdalsassbdalsass self-assigned thisMar 18, 2025
@steffunky
Copy link
Member

A more elegant way would be to undo your changes and modify how thechange listener is added injs/links/links_widget.js.
Line 393 we could remove the listener from the input element itself (that selectize removes on initialization) and put it on the body with a selector that matches our input.

The code would change from this:

$('#linkedset_'+me.id+' :input[name^="attr_'+me.sAttCode+'["]').off('change').on('change', function () {    ...});

to this:

$('body').off('change', '#linkedset_'+me.id+' :input[name^="attr_'+me.sAttCode+'["]')    .on('change', '#linkedset_'+me.id+' :input[name^="attr_'+me.sAttCode+'["]', function () {.....});

I tested it and it seems to work well

rquetiez reacted with thumbs up emoji

@jf-cbdjf-cbdforce-pushed thesupport/3.2.1 branch 3 times, most recently frome5198dd tob38bf76CompareMarch 21, 2025 16:58
@steffunkysteffunky merged commitf44468b intosupport/3.2.1Mar 24, 2025
@steffunkysteffunky deleted the feature/8245-indirect-link-external-key-not-saved branchMarch 24, 2025 13:05
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@eespieeespieAwaiting requested review from eespie

@rquetiezrquetiezAwaiting requested review from rquetiez

@jf-cbdjf-cbdAwaiting requested review from jf-cbd

Assignees

@bdalsassbdalsass

Labels
bugSomething isn't workinginternalWork made by Combodo
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@bdalsass@steffunky

[8]ページ先頭

©2009-2025 Movatter.jp