When deleting a user fails, the alert dialog did not pop up properly

pangbo · March 10, 2023, 1:14pm

In my recent test, I found that when the user fails to delete himself in the “Preferences”, the corresponding error dialog box does not pop up normally.

After reading the source code, I suspect that this line of code is missing a call to next.

github.com

discourse/discourse/blob/3e8d3494651c50751ec248506a9e4a13137e5c6b/app/assets/javascripts/discourse/app/controllers/preferences/account.js#L219


      
                        this.dialog.alert({
                          message: I18n.t("user.deleted_yourself"),
                          didConfirm: () =>
                            DiscourseURL.redirectAbsolute(getURL("/")),
                          didCancel: () =>
                            DiscourseURL.redirectAbsolute(getURL("/")),
                        });
                      });
                    },
                    () => {
                      this.dialog.alert(I18n.t("user.delete_yourself_not_allowed"));
                      this.set("deleting", false);
                    }
                  );
                },
              },
              {
                label: I18n.t("composer.cancel"),
              },
            ],
          });

I modified this line in development environment, like this
next(() => this.dialog.alert(I18n.t("user.delete_yourself_not_allowed")));
then I found that the dialog popped up normally.

I’m not very familiar with ember, so I don’t know if this modification is correct, but it seems that there must be a bug here

Topic		Replies	Views
Delete user feature stopped working on staging/production docker setups, but works on local (no docker setup) installation	16	318	February 5, 2023
Removal of ember-addons/ember-computed-decorators dev	12	1489	February 20, 2020
Alert used to Notify AutoComplete Support? dev	5	1094	February 23, 2016
Could not find module ember addons support	4	672	February 10, 2022
Having some issues with our plugin and Discourse's Ember API. Can anyone help? dev ember , rest-api	5	237	March 29, 2024

When deleting a user fails, the alert dialog did not pop up properly

Related Topics