jenkins-bot has submitted this change and it was merged.

Change subject: Track the display of render pages
......................................................................


Track the display of render pages

This counts the number of each type of render
page that is shown to a user.

Change-Id: I6f965adb701e3a5ec7b5542e414e1790c44be0f8
---
M Collection.body.php
1 file changed, 5 insertions(+), 0 deletions(-)

Approvals:
  Legoktm: Looks good to me, approved
  Thiemo Mättig (WMDE): Looks good to me, but someone else must approve
  jenkins-bot: Verified



diff --git a/Collection.body.php b/Collection.body.php
index 77ec9da..5e0ebf4 100644
--- a/Collection.body.php
+++ b/Collection.body.php
@@ -1033,6 +1033,7 @@
                $this->setHeaders();
                $request = $this->getRequest();
                $out = $this->getOutput();
+               $stats = $out->getStats();
 
                $collectionId = $request->getVal( 'collection_id' );
                $writer = $request->getVal( 'writer' );
@@ -1078,6 +1079,7 @@
                        }
                        $template->set( 'progress', $progress );
                        $out->addTemplate( $template );
+                       $stats->increment( 'collection.renderingpage.pending' );
                        break;
                case 'finished':
                        $out->setPageTitle( $this->msg( 
'coll-rendering_finished_title' ) );
@@ -1089,6 +1091,7 @@
                        $template->set( 'query', $query );
                        $template->set( 'return_to', $return_to );
                        $out->addTemplate( $template );
+                       $stats->increment( 'collection.renderingpage.finished' 
);
                        break;
                case 'failed':
                        $out->setPageTitle( $this->msg( 
'coll-rendering_failed_title' ) );
@@ -1105,8 +1108,10 @@
                        $template->set( 'query', $query );
                        $template->set( 'return_to', $return_to );
                        $out->addTemplate( $template );
+                       $stats->increment( 'collection.renderingpage.failed' );
                        break;
                default:
+                       $stats->increment( 'collection.renderingpage.unknown' );
                        throw new Exception( __METHOD__ . "(): unknown state 
'{$result->get( 'state' )}'");
                }
        }

-- 
To view, visit https://gerrit.wikimedia.org/r/303814
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I6f965adb701e3a5ec7b5542e414e1790c44be0f8
Gerrit-PatchSet: 3
Gerrit-Project: mediawiki/extensions/Collection
Gerrit-Branch: master
Gerrit-Owner: Addshore <[email protected]>
Gerrit-Reviewer: Cscott <[email protected]>
Gerrit-Reviewer: Legoktm <[email protected]>
Gerrit-Reviewer: Thiemo Mättig (WMDE) <[email protected]>
Gerrit-Reviewer: WMDE-Fisch <[email protected]>
Gerrit-Reviewer: WMDE-jand <[email protected]>
Gerrit-Reviewer: WMDE-leszek <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to