I’m writing a plugin to extend badge functionality. When the “save” button is pressed on the admin badge page, I want to execute some additional code. Discourse Javascript: app/assets/javascripts/admin/addon/controllers/admin-badges/show.js d9affeca0 this.buffered.set("image_url", null); } @action showPreview(badge, explain, event) { event?.preventDefault(); this.send("preview", badge, explain); } @action save() { if (!this.saving) { let fields = [ "allow_title", "multiple_grant", "listable", "auto_revoke", "enabled", "show_posts", "target_posts", "name", Discourse hbs: app/assets/javascripts/admin/addon/templates/admin-badges/show.hbs d9affeca0 @checked={{this.buffered.show_posts}} disabled={{this.readOnly}} /> {{i18n "admin.badges.show_posts"}} {{this.savingStatus}} {{#unless this.readOnly}} { api.modifyClass('controller:admin-badges/show', { pluginId: 'discourse-badge-extension', actions: { save() { console.log("Custom pre-save logic triggered"); alert("Pre-save logic"); this._super(...arguments); // Call the original save method console.log("Post-save logic triggered"); alert("Post-save logic"); }, } }); alert("Hello! I am an alert box!!"); }); } }; It’s an initializer. The alert box "Hello! I am an alert box!!" is triggered, indicating the code is being run on reload. Neither the console.log lines nor the alerts run on saving, indicating the overriding is not working? the controller name does not give an error controller:admin-badges/show (but intentionally adding a typo does, suggesting I am targeting a real controller) It should be easy to repo given it’s just one initializer. What am I doing wrong???

Hey @piffy. Unfortunately this is an incompatibility with JS ‘Native class syntax’, EmberObjects’s reopen() functionality, and the @action decorator (which technically, creates a native JS ‘getter’). I’m afraid I don’t have an immediate suggestion to get this particular override working, but we are working on a more robust solution for the future. Depending on what you’re trying to do, maybe hooking into the badge model’s “save” function could work? app/assets/javascripts/discourse/app/models/badge.js d9affeca0 }); } }, @discourseComputed("badge_type.name") badgeTypeClassName(type) { type = type || ""; return `badge-type-${type.toLowerCase()}`; }, save(data) { let url = "/admin/badges", type = "POST"; if (this.id) { // We are updating an existing badge. url += `/${this.id}`; type = "PUT"; } return ajax(url, { type, data }).then((json) => {

Having a hard time overriding the save() function on the admin badge page

Development

piffy January 8, 2024, 8:53pm 3

Thank you for the fast and comprehensive answer. I will look into that other save function.

Topic		Replies	Views
How to bind custom fields and sync form data on admin badge page Development badges	1	109	May 15, 2025
Request for a plugin outlet on admin badge page Development badges	2	505	January 9, 2024
Recommended replacement for modifyClass("model:composer") when overriding save() Development composer	7	351	August 21, 2026
modifyClass does not override action? Development	4	355	February 28, 2024
Overwriting controllers in plugin Development	2	3084	March 7, 2016

Having a hard time overriding the save() function on the admin badge page

Related topics