discourse
/
discourse-adplugin
mirror of https://github.com/discourse/discourse-adplugin.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

FIX: HouseAdsChooser `onChange` handling 

Having an action named the same as an argument is no longer possible in a classic component. Move the logic to the parent instead.

Followup to 7685ebf396
This commit is contained in:
David Taylor 2024-12-06 10:05:55 +00:00
parent 00298bf01a
commit 11730caf74
No known key found for this signature in database
GPG Key ID: 46904C18B1D3F434
3 changed files with 9 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
admin/assets/javascripts/discourse/components/house-ads-chooser.js
View File

@ -22,11 +22,6 @@ export default class HouseAdsChooser extends MultiSelectComponent {
.filter(Boolean);
}
// TODO: kept for legacy, remove when Discourse is 2.5
mutateValues(values) {
this.set("settingValue", values.join(this.tokenSeparator));
}
computeValues() {
return this.settingValue.split(this.tokenSeparator).filter(Boolean);
}
@ -35,10 +30,4 @@ export default class HouseAdsChooser extends MultiSelectComponent {
get content() {
return makeArray(this.choices);
}
@action
onChange(value) {
const settingValue = makeArray(value).join(this.tokenSeparator);
this.onChange?.(settingValue);
}
}

2
admin/assets/javascripts/discourse/components/house-ads-list-setting.hbs
View File

@ -2,7 +2,7 @@
{{house-ads-chooser
settingValue=this.adValue
choices=this.adNames
onChange=(action (mut this.adValue))
onChange=this.changeAdValue
}}
<div class="setting-controls">
{{#if this.changed}}

8
admin/assets/javascripts/discourse/components/house-ads-list-setting.js
View File

@ -1,8 +1,16 @@
import { action } from "@ember/object";
import { mapBy } from "@ember/object/computed";
import { classNames } from "@ember-decorators/component";
import { makeArray } from "discourse-common/lib/helpers";
import HouseAdsSetting from "discourse/plugins/discourse-adplugin/discourse/components/house-ads-setting";
@classNames("house-ads-setting house-ads-list-setting")
export default class HouseAdsListSetting extends HouseAdsSetting {
@mapBy("allAds", "name") adNames;
@action
changeAdValue(value) {
const settingValue = makeArray(value).join("|");
this.set("adValue", settingValue);
}
}