fix after merge

Author: agoujot

src/Controller/PagesController.php

@@ -82,11 +82,11 @@ public function indexAction(): Response {
 	 * @codeCoverageIgnore
 	 */
 	protected function setUpPages(
-    PagesRepository $pagesRepo,
-    string $redirects,
-    string $deleted,
-    bool $countsOnly = false
-  ): Pages {
+		PagesRepository $pagesRepo,
+		string $redirects,
+		string $deleted,
+		bool $countsOnly = false
+	): Pages {
 		if ( $this->user->isIpRange() ) {
 			$this->params['username'] = $this->user->getUsername();
 			$this->throwXtoolsException( $this->getIndexRoute(), 'error-ip-range-unsupported' );
@@ -102,7 +102,7 @@ protected function setUpPages(
 			$this->start,
 			$this->end,
 			$this->offset,
-      $countsOnly
+			$countsOnly
 		);
 	}
 
@@ -134,11 +134,11 @@ public function resultAction(
 		string $redirects = Pages::REDIR_NONE,
 		string $deleted = Pages::DEL_ALL
 	): RedirectResponse|Response {
-    $countsOnly = filter_var(
-			$this->request->query->get('countsOnly', 'false'),
+		$countsOnly = filter_var(
+			$this->request->query->get( 'countsOnly', 'false' ),
 			FILTER_VALIDATE_BOOLEAN,
 		) && (
-			$this->request->query->get('format', 'html') === 'html'
+			$this->request->query->get( 'format', 'html' ) === 'html'
 		);
 		// Check for legacy values for 'redirects', and redirect
 		// back with correct values if need be. This could be refactored
@@ -150,7 +150,7 @@ public function resultAction(
 				'redirects' => Pages::REDIR_NONE,
 				'deleted' => $deleted,
 				'offset' => $this->offset,
-        'countsOnly' => $countsOnly,
+				'countsOnly' => $countsOnly,
 			] ) );
 		}
 

src/Controller/XtoolsController.php

@@ -638,7 +638,7 @@ public function getParams(): array {
 			'include_pattern',
 			'exclude_pattern',
 			'classonly',
-      'countsOnly',
+			'countsOnly',
 
 			// Legacy parameters.
 			'user',

src/Model/Pages.php

@@ -33,8 +33,8 @@ class Pages extends Model {
 
 	/** @var array Number of redirects/pages that were created/deleted, broken down by namespace. */
 	protected array $countsByNamespace;
-  
-  /** @var bool Whether to only get the counts */
+
+	/** @var bool Whether to only get the counts */
 	protected bool $countsOnly;
 
 	/**
@@ -48,7 +48,7 @@ class Pages extends Model {
 	 * @param int|false $start Start date as Unix timestamp.
 	 * @param int|false $end End date as Unix timestamp.
 	 * @param int|false $offset Unix timestamp. Used for pagination.
-   * @param bool $countsOnly Whether to only get the counts
+	 * @param bool $countsOnly Whether to only get the counts
 	 */
 	public function __construct(
 		protected Repository|PagesRepository $repository,
@@ -60,11 +60,12 @@ public function __construct(
 		protected int|false $start = false,
 		protected int|false $end = false,
 		protected int|false $offset = false,
-    protected bool $countsOnly = false,
+		bool $countsOnly = false,
 	) {
 		$this->namespace = $namespace === 'all' ? 'all' : (int)$namespace;
 		$this->redirects = $redirects ?: self::REDIR_NONE;
 		$this->deleted = $deleted ?: self::DEL_ALL;
+		$this->countsOnly = $countsOnly;
 	}
 
 	/**
@@ -83,6 +84,14 @@ public function getDeleted(): string {
 		return $this->deleted;
 	}
 
+	/**
+	 * Whether to only calculate counts.
+	 * @return bool
+	 */
+	public function isCountsOnly(): bool {
+		return $this->countsOnly;
+	}
+
 	/**
 	 * Fetch and prepare the pages created by the user.
 	 * @param bool $all Whether to get *all* results. This should only be used for
@@ -93,7 +102,7 @@ public function getDeleted(): string {
 	public function prepareData( bool $all = false ): array {
 		$this->pages = [];
 
-		if ($this->countsOnly) {
+		if ( $this->countsOnly ) {
 			return [];
 		}
 

templates/pages/result.html.twig

@@ -238,10 +238,9 @@
         {{ layout.content_block('summary', content, downloadLink) }}
 
         {################## LIST OF PAGES ###################}
-        {# not if countsOnly is true #}
-        {% if not countsOnly %}
+        {% if not pages.countsOnly %}
             {% include 'pages/_pages_list.html.twig' with {'pages': pages} %}
-    
+
             <div class="text-muted times-in-utc" style="clear:both">
                 {{ msg('times-in-utc') }}
         {% endif %}