IntenseWebs/discourse
3
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2024-11-26 19:00:32 -06:00
Code Issues Packages Projects Releases Wiki Activity

UX: Don't display all groups option if group directory is disabled.a

Browse Source 
https://meta.discourse.org/t/all-groups-option-leads-to-access-denied-page/88464
This commit is contained in:
Guo Xiang Tan 2018-05-28 11:32:55 +08:00
parent 857a4e0006
commit 0b55416742
4 changed files with 52 additions and 42 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
app/assets/javascripts/discourse/templates/groups/index.hbs
View File

@ -9,7 +9,7 @@
<div class="groups-header-filters">
{{text-field value=filterInput
placeholderKey="groups.index.all_groups"
placeholderKey="groups.index.all"
class="groups-header-filters-name no-blur"}}
{{combo-box value=type

14
app/assets/javascripts/select-kit/components/group-dropdown.js.es6
View File

@ -29,11 +29,15 @@ export default ComboBoxComponent.extend({
@computed
collectionHeader() {
return `
<a href="${Discourse.getURL("/groups")}" class="group-dropdown-filter">
${I18n.t("groups.index.all").toLowerCase()}
</a>
`.htmlSafe();
if (this.siteSettings.enable_group_directory ||
(this.currentUser && this.currentUser.get('staff'))) {
return `
<a href="${Discourse.getURL("/groups")}" class="group-dropdown-filter">
${I18n.t("groups.index.all").toLowerCase()}
</a>
`.htmlSafe();
}
},
actions: {

1
config/locales/client.en.yml
View File

@ -464,7 +464,6 @@ en:
all: "All Groups"
empty: "There are no visible groups."
filter: "Filter by group type"
all_groups: "All Groups"
owner_groups: "Groups I am an owner of"
close_groups: "Closed Groups"
automatic_groups: "Automatic Groups"

77
test/javascripts/acceptance/group-test.js.es6
View File

@ -1,6 +1,10 @@
import { acceptance, logIn } from "helpers/qunit-helpers";
acceptance("Group");
acceptance("Group", {
settings: {
enable_group_directory: true
}
});
const response = object => {
return [
@ -10,50 +14,53 @@ const response = object => {
];
};
QUnit.test("Anonymous Viewing Group", assert => {
visit("/groups/discourse");
QUnit.test("Anonymous Viewing Group", async assert => {
await visit("/groups/discourse");
andThen(() => {
assert.equal(
count(".nav-pills li a[title='Messages']"),
0,
'it deos not show group messages navigation link'
);
});
assert.equal(
count(".nav-pills li a[title='Messages']"),
0,
'it deos not show group messages navigation link'
);
click(".nav-pills li a[title='Activity']");
await click(".nav-pills li a[title='Activity']");
andThen(() => {
assert.ok(count('.user-stream-item') > 0, "it lists stream items");
});
assert.ok(count('.user-stream-item') > 0, "it lists stream items");
click(".activity-nav li a[href='/groups/discourse/activity/topics']");
await click(".activity-nav li a[href='/groups/discourse/activity/topics']");
andThen(() => {
assert.ok(find('.topic-list'), "it shows the topic list");
assert.equal(count('.topic-list-item'), 2, "it lists stream items");
});
assert.ok(find('.topic-list'), "it shows the topic list");
assert.equal(count('.topic-list-item'), 2, "it lists stream items");
click(".activity-nav li a[href='/groups/discourse/activity/mentions']");
await click(".activity-nav li a[href='/groups/discourse/activity/mentions']");
andThen(() => {
assert.ok(count('.user-stream-item') > 0, "it lists stream items");
});
assert.ok(count('.user-stream-item') > 0, "it lists stream items");
assert.ok(find(".nav-pills li a[title='Edit Group']").length === 0, 'it should not show messages tab if user is not admin');
assert.ok(find(".nav-pills li a[title='Logs']").length === 0, 'it should not show Logs tab if user is not admin');
assert.ok(count('.user-stream-item') > 0, "it lists stream items");
andThen(() => {
assert.ok(find(".nav-pills li a[title='Edit Group']").length === 0, 'it should not show messages tab if user is not admin');
assert.ok(find(".nav-pills li a[title='Logs']").length === 0, 'it should not show Logs tab if user is not admin');
assert.ok(count('.user-stream-item') > 0, "it lists stream items");
});
await expandSelectKit('.group-dropdown');
selectKit('.group-dropdown').expand();
assert.equal(
find('.select-kit-row').text().trim(), 'discourse',
'it displays the right row'
);
andThen(() => {
assert.equal(
find('.select-kit-row').text().trim(), 'discourse',
'it displays the right row'
);
});
assert.equal(
find('.group-dropdown-filter').text().trim(), I18n.t("groups.index.all").toLowerCase(),
'it displays the right header'
);
Discourse.SiteSettings.enable_group_directory = false;
await visit("/groups");
await visit("/groups/discourse");
await expandSelectKit('.group-dropdown');
assert.equal(
find('.group-dropdown-filter').length, 0,
'it should not display the default header'
);
});
QUnit.test("Anonymous Viewing Automatic Group", assert => {