Show login modal if replying as guest; disable button if no permission

Also hide the first item in dropdown-split menus
This commit is contained in:
Toby Zerner
2015-05-04 11:08:45 +09:30
parent 538784744c
commit ae2930dc35
3 changed files with 38 additions and 11 deletions

View File

@ -13,10 +13,11 @@ export default class DropdownSplit extends Component {
var firstItem = this.props.items[0];
var items = listItems(this.props.items);
var buttonProps = { className: this.props.buttonClass || 'btn btn-default' };
var buttonProps = {};
for (var i in firstItem.props) {
buttonProps[i] = firstItem.props[i];
}
buttonProps.className = (buttonProps.className || '')+' '+(this.props.buttonClass || 'btn btn-default');
return m('div', {className: 'dropdown dropdown-split btn-group item-count-'+(items.length)+' '+this.props.className}, [
ActionButton.component(buttonProps),

View File

@ -5,6 +5,7 @@ import DiscussionPage from 'flarum/components/discussion-page';
import ActionButton from 'flarum/components/action-button';
import Separator from 'flarum/components/separator';
import ComposerReply from 'flarum/components/composer-reply';
import LoginModal from 'flarum/components/login-modal';
class Discussion extends Model {
unreadCount() {
@ -23,7 +24,10 @@ class Discussion extends Model {
var items = new ItemList();
if (context instanceof DiscussionPage) {
items.add('reply', ActionButton.component({ icon: 'reply', label: 'Reply', onclick: this.replyAction.bind(this) }));
items.add('reply', !app.session.user() || this.canReply()
? ActionButton.component({ icon: 'reply', label: app.session.user() ? 'Reply' : 'Log In to Reply', onclick: this.replyAction.bind(this) })
: ActionButton.component({ icon: 'reply', label: 'Can\'t Reply', className: 'disabled', title: 'You don\'t have permission to reply to this discussion.' })
);
items.add('separator', Separator.component());
}
@ -40,7 +44,7 @@ class Discussion extends Model {
}
replyAction() {
if (app.session.user()) {
if (app.session.user() && this.canReply()) {
if (app.current.discussion && app.current.discussion().id() === this.id()) {
app.current.streamContent.goToLast();
}
@ -49,8 +53,11 @@ class Discussion extends Model {
discussion: this
}));
app.composer.show();
} else {
// signup
} else if (!app.session.user()) {
app.modal.show(new LoginModal({
message: 'You must be logged in to do that.',
callback: this.replyAction.bind(this)
}));
}
}