jenkins-bot has submitted this change and it was merged.
Change subject: Add action object for no-js support
......................................................................
Add action object for no-js support
Change-Id: Ia3cc3c3b51a5183ea8ba05f7ee9422239471f3e3
---
M Flow.php
M FlowActions.php
M includes/Actions/CloseTopicAction.php
A includes/Actions/CompareHeaderRevisionsAction.php
A includes/Actions/ComparePostRevisionsAction.php
A includes/Actions/ComparePostSummaryRevisionsAction.php
A includes/Actions/PostSingleViewAction.php
A includes/Actions/ViewHeaderAction.php
A includes/Actions/ViewTopicSummaryAction.php
9 files changed, 106 insertions(+), 3 deletions(-)
Approvals:
EBernhardson: Looks good to me, approved
jenkins-bot: Verified
diff --git a/Flow.php b/Flow.php
index 33b10a7..51bd3e4 100755
--- a/Flow.php
+++ b/Flow.php
@@ -247,6 +247,12 @@
$wgAutoloadClasses['Flow\Actions\ViewAction'] = $dir .
'includes/Actions/ViewAction.php';
$wgAutoloadClasses['Flow\Actions\ReplyAction'] = $dir .
'includes/Actions/ReplyAction.php';
$wgAutoloadClasses['Flow\Actions\HistoryAction'] = $dir .
'includes/Actions/HistoryAction.php';
+$wgAutoloadClasses['Flow\Actions\ComparePostSummaryRevisionsAction'] = $dir .
'includes/Actions/ComparePostSummaryRevisionsAction.php';
+$wgAutoloadClasses['Flow\Actions\CompareHeaderRevisionsAction'] = $dir .
'includes/Actions/CompareHeaderRevisionsAction.php';
+$wgAutoloadClasses['Flow\Actions\ViewHeaderAction'] = $dir .
'includes/Actions/ViewHeaderAction.php';
+$wgAutoloadClasses['Flow\Actions\ComparePostRevisionsAction'] = $dir .
'includes/Actions/ComparePostRevisionsAction.php';
+$wgAutoloadClasses['Flow\Actions\PostSingleViewAction'] = $dir .
'includes/Actions/PostSingleViewAction.php';
+$wgAutoloadClasses['Flow\Actions\ViewTopicSummaryAction'] = $dir .
'includes/Actions/ViewTopicSummaryAction.php';
// phpunit helper
$wgAutoloadClasses['Flow\Tests\FlowTestCase'] = $dir .
'tests/FlowTestCase.php';
diff --git a/FlowActions.php b/FlowActions.php
index 491f49f..3dd5388 100644
--- a/FlowActions.php
+++ b/FlowActions.php
@@ -95,7 +95,7 @@
),
'class' => 'flow-history-create-topic-summary',
),
- 'handler-class' => 'Flow\Actions\CreateTopicAction',
+ 'handler-class' => 'Flow\Actions\EditTopicSummaryAction',
),
'edit-topic-summary' => array(
@@ -121,7 +121,7 @@
),
'class' => 'flow-history-edit-topic-summary',
),
- 'handler-class' => 'Flow\Actions\EditTopicAction',
+ 'handler-class' => 'Flow\Actions\EditTopicSummaryAction',
),
'edit-title' => array(
@@ -610,6 +610,31 @@
'handler-class' => 'Flow\Actions\HistoryAction',
),
+ // Actions not tied to a particular revision change_type
+ // or just move these to a different file
+ 'close-open-topic' => array(
+ 'handler-class' => 'Flow\Actions\CloseTopicAction',
+ ),
+ 'compare-header-revisions' => array(
+ 'handler-class' => 'Flow\Actions\CompareHeaderRevisionsAction',
+ ),
+ 'view-header' => array(
+ 'handler-class' => 'Flow\Actions\ViewHeaderAction',
+ ),
+ 'compare-post-revisions' => array(
+ 'handler-class' => 'Flow\Actions\ComparePostRevisionsAction',
+ ),
+ // @todo - This is a very bad action name, consolidate with view-post
action
+ 'single-view' => array(
+ 'handler-class' => 'Flow\Actions\PostSingleViewAction',
+ ),
+ 'view-topic-summary' => array(
+ 'handler-class' => 'Flow\Actions\ViewTopicSummaryAction',
+ ),
+ 'compare-postsummary-revisions' => array(
+ 'handler-class' =>
'Flow\Actions\ComparePostSummaryRevisionsAction',
+ ),
+
// log & all other formatters have same config as history
'log' => 'history',
'recentchanges' => 'history',
diff --git a/includes/Actions/CloseTopicAction.php
b/includes/Actions/CloseTopicAction.php
index 635312d..f89aebd 100644
--- a/includes/Actions/CloseTopicAction.php
+++ b/includes/Actions/CloseTopicAction.php
@@ -7,6 +7,6 @@
class CloseTopicAction extends FlowAction {
function __construct( Page $page, IContextSource $context ) {
- parent::__construct( $page, $context, 'close-topic' );
+ parent::__construct( $page, $context, 'close-open-topic' );
}
}
\ No newline at end of file
diff --git a/includes/Actions/CompareHeaderRevisionsAction.php
b/includes/Actions/CompareHeaderRevisionsAction.php
new file mode 100644
index 0000000..bd01a60
--- /dev/null
+++ b/includes/Actions/CompareHeaderRevisionsAction.php
@@ -0,0 +1,12 @@
+<?php
+
+namespace Flow\Actions;
+
+use IContextSource;
+use Page;
+
+class CompareHeaderRevisionsAction extends FlowAction {
+ function __construct( Page $page, IContextSource $context ) {
+ parent::__construct( $page, $context,
'compare-header-revisions' );
+ }
+}
diff --git a/includes/Actions/ComparePostRevisionsAction.php
b/includes/Actions/ComparePostRevisionsAction.php
new file mode 100644
index 0000000..39a0740
--- /dev/null
+++ b/includes/Actions/ComparePostRevisionsAction.php
@@ -0,0 +1,12 @@
+<?php
+
+namespace Flow\Actions;
+
+use IContextSource;
+use Page;
+
+class ComparePostRevisionsAction extends FlowAction {
+ function __construct( Page $page, IContextSource $context ) {
+ parent::__construct( $page, $context, 'compare-post-revisions'
);
+ }
+}
diff --git a/includes/Actions/ComparePostSummaryRevisionsAction.php
b/includes/Actions/ComparePostSummaryRevisionsAction.php
new file mode 100644
index 0000000..e40e277
--- /dev/null
+++ b/includes/Actions/ComparePostSummaryRevisionsAction.php
@@ -0,0 +1,12 @@
+<?php
+
+namespace Flow\Actions;
+
+use IContextSource;
+use Page;
+
+class ComparePostSummaryRevisionsAction extends FlowAction {
+ function __construct( Page $page, IContextSource $context ) {
+ parent::__construct( $page, $context,
'compare-postsummary-revisions' );
+ }
+}
diff --git a/includes/Actions/PostSingleViewAction.php
b/includes/Actions/PostSingleViewAction.php
new file mode 100644
index 0000000..32666d8
--- /dev/null
+++ b/includes/Actions/PostSingleViewAction.php
@@ -0,0 +1,12 @@
+<?php
+
+namespace Flow\Actions;
+
+use IContextSource;
+use Page;
+
+class PostSingleViewAction extends FlowAction {
+ function __construct( Page $page, IContextSource $context ) {
+ parent::__construct( $page, $context, 'single-view' );
+ }
+}
diff --git a/includes/Actions/ViewHeaderAction.php
b/includes/Actions/ViewHeaderAction.php
new file mode 100644
index 0000000..7ecf6ee
--- /dev/null
+++ b/includes/Actions/ViewHeaderAction.php
@@ -0,0 +1,12 @@
+<?php
+
+namespace Flow\Actions;
+
+use IContextSource;
+use Page;
+
+class ViewHeaderAction extends FlowAction {
+ function __construct( Page $page, IContextSource $context ) {
+ parent::__construct( $page, $context, 'view-header' );
+ }
+}
diff --git a/includes/Actions/ViewTopicSummaryAction.php
b/includes/Actions/ViewTopicSummaryAction.php
new file mode 100644
index 0000000..7eba3bc
--- /dev/null
+++ b/includes/Actions/ViewTopicSummaryAction.php
@@ -0,0 +1,12 @@
+<?php
+
+namespace Flow\Actions;
+
+use IContextSource;
+use Page;
+
+class ViewTopicSummaryAction extends FlowAction {
+ function __construct( Page $page, IContextSource $context ) {
+ parent::__construct( $page, $context, 'view-topic-summary' );
+ }
+}
--
To view, visit https://gerrit.wikimedia.org/r/143082
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia3cc3c3b51a5183ea8ba05f7ee9422239471f3e3
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Flow
Gerrit-Branch: master
Gerrit-Owner: Bsitu <[email protected]>
Gerrit-Reviewer: EBernhardson <[email protected]>
Gerrit-Reviewer: SG <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
