github-mirror/discourse
2
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2024-11-25 18:03:43 +08:00
Code Issues Actions 6 Packages Projects Releases Wiki Activity

FIX: Keep category name in URL when filtering (#10317)

Browse Source
This commit is contained in:
Bianca Nenciu 2020-07-29 14:57:05 +03:00 committed by GitHub
parent d2380a01b6
commit fd02856f80
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 13 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
app/assets/javascripts/admin/controllers/admin-site-settings.js
View File

@ -105,7 +105,7 @@ export default Controller.extend({
if (this._skipBounce) {
this.set("_skipBounce", false);
} else {
this.filterContentNow();
this.filterContentNow(this.categoryNameKey);
}
}, INPUT_DELAY),

19
test/javascripts/acceptance/admin-site-settings-test.js
View File

@ -10,7 +10,7 @@ acceptance("Admin - Site Settings", {
},
pretend(server, helper) {
server.put("/admin/site_settings/title", (body) => {
server.put("/admin/site_settings/title", body => {
titleOverride = body.requestBody.split("=")[1];
return helper.response({ success: "OK" });
});
@ -22,14 +22,14 @@ acceptance("Admin - Site Settings", {
titleSetting.value = titleOverride;
}
const response = {
site_settings: [titleSetting, ...fixtures.slice(1)],
site_settings: [titleSetting, ...fixtures.slice(1)]
};
return helper.response(response);
});
},
}
});
QUnit.test("upload site setting", async (assert) => {
QUnit.test("upload site setting", async assert => {
await visit("/admin/site_settings");
assert.ok(
@ -40,7 +40,7 @@ QUnit.test("upload site setting", async (assert) => {
assert.ok(exists(".row.setting.upload .undo"), "undo button is present");
});
QUnit.test("changing value updates dirty state", async (assert) => {
QUnit.test("changing value updates dirty state", async assert => {
await visit("/admin/site_settings");
await fillIn("#setting-filter", " title ");
assert.equal(count(".row.setting"), 1, "filter returns 1 site setting");
@ -89,7 +89,7 @@ QUnit.test("changing value updates dirty state", async (assert) => {
QUnit.test(
"always shows filtered site settings if a filter is set",
async (assert) => {
async assert => {
await visit("/admin/site_settings");
await fillIn("#setting-filter", "title");
assert.equal(count(".row.setting"), 1);
@ -104,7 +104,7 @@ QUnit.test(
}
);
QUnit.test("filter settings by plugin name", async (assert) => {
QUnit.test("filter settings by plugin name", async assert => {
await visit("/admin/site_settings");
await fillIn("#setting-filter", "plugin:discourse-logo");
@ -114,3 +114,8 @@ QUnit.test("filter settings by plugin name", async (assert) => {
await fillIn("#setting-filter", "plugin:discourse-plugin");
assert.equal(count(".row.setting"), 0);
});
QUnit.test("category name is preserved", async assert => {
await visit("admin/site_settings/category/login?filter=test");
assert.equal(currentURL(), "admin/site_settings/category/login?filter=test");
});