';
+ // Long topics are printed a page at a time.
+ if (isset(Utils::$context['page_index'])) {
+ echo '
+
', Utils::$context['page_index'], '
';
+ }
+
// Which option is set, text or text&images
if (isset($_REQUEST['images'])) {
echo '
diff --git a/tests/Integration/Http/TopicPrintTest.php b/tests/Integration/Http/TopicPrintTest.php
new file mode 100644
index 0000000000..b36bc2d92c
--- /dev/null
+++ b/tests/Integration/Http/TopicPrintTest.php
@@ -0,0 +1,187 @@
+seedTopic();
+
+ $first = $this->fetch('?action=printpage;topic=' . $topic_id . '.0');
+
+ $this->assertSame(
+ self::PER_PAGE,
+ $first->xpath('//div[@class="postheader"]')->length,
+ 'the print page rendered the whole topic instead of one page of it',
+ );
+
+ $this->assertStringNotContainsString(
+ $this->bodyOfPost(self::POSTS),
+ $first->text(),
+ 'the last post of the topic is on the first print page',
+ );
+
+ $this->assertNoErrorsLogged('printing a topic logged something.' . "\n");
+ }
+
+ public function testTheRestOfTheTopicIsOnTheFollowingPages(): void
+ {
+ $topic_id = $this->seedTopic();
+
+ $last = $this->fetch('?action=printpage;topic=' . $topic_id . '.' . (self::POSTS - 1));
+
+ $this->assertStringContainsString(
+ $this->bodyOfPost(self::POSTS),
+ $last->text(),
+ 'the last post of the topic cannot be printed at all',
+ );
+
+ $this->assertNoErrorsLogged('printing the last page logged something.' . "\n");
+ }
+
+ /**
+ * Without these the rest of a long topic is unreachable, since the print
+ * page is the only thing that links to itself.
+ */
+ public function testAPrintPageLinksToTheOtherPages(): void
+ {
+ $topic_id = $this->seedTopic();
+
+ $first = $this->fetch('?action=printpage;topic=' . $topic_id . '.0');
+
+ $this->assertGreaterThan(
+ 0,
+ $first->xpath(
+ '//a[contains(@href, "action=printpage")][contains(@href, "topic=' . $topic_id . '.' . self::PER_PAGE . '")]',
+ )->length,
+ 'the print page does not link to its second page',
+ );
+ }
+
+ /******************
+ * Internal methods
+ ******************/
+
+ protected function setUp(): void
+ {
+ parent::setUp();
+
+ // A print page shows no more posts than the "All" view is allowed to,
+ // so this is what decides where it stops.
+ $this->previous_max = $this->rawSetting('enableAllMessages');
+
+ Config::updateModSettings(['enableAllMessages' => self::PER_PAGE]);
+ }
+
+ protected function tearDown(): void
+ {
+ // Before the parent runs, while the connection is still ours.
+ Config::updateModSettings(['enableAllMessages' => (int) $this->previous_max]);
+
+ if ($this->created_topics !== []) {
+ Topic::remove($this->created_topics);
+
+ $this->created_topics = [];
+ }
+
+ parent::tearDown();
+ }
+
+ /**
+ * Starts a topic holding more posts than one print page may show.
+ *
+ * @return int The new topic's id.
+ */
+ private function seedTopic(): int
+ {
+ $this->actingAs($this->adminId());
+
+ $topic_id = 0;
+
+ for ($i = 1; $i <= self::POSTS; $i++) {
+ $msgOptions = [
+ 'subject' => ($i === 1 ? '' : 'Re: ') . 'Print pagination',
+ 'body' => $this->bodyOfPost($i),
+ 'send_notifications' => false,
+ ];
+ $topicOptions = [
+ 'id' => $topic_id,
+ 'board' => 1,
+ ];
+ $posterOptions = [
+ 'id' => $this->adminId(),
+ ];
+
+ Msg::create($msgOptions, $topicOptions, $posterOptions);
+
+ $topic_id = (int) $topicOptions['id'];
+
+ if ($this->created_topics === []) {
+ $this->created_topics[] = $topic_id;
+ }
+ }
+
+ return $topic_id;
+ }
+
+ /**
+ * The body of one of the seeded posts.
+ *
+ * @param int $number Which post.
+ * @return string Its body.
+ */
+ private function bodyOfPost(int $number): string
+ {
+ return 'Print pagination post number ' . $number . '.';
+ }
+}
diff --git a/tests/Unit/TopicPrintTest.php b/tests/Unit/TopicPrintTest.php
new file mode 100644
index 0000000000..dcc6dd0ee8
--- /dev/null
+++ b/tests/Unit/TopicPrintTest.php
@@ -0,0 +1,74 @@
+assertSame(TopicPrint::DEFAULT_MAX_POSTS, $this->getPostsPerPage());
+ }
+
+ // Note: the section banner above must not be the first thing in this group when
+ // the first member carries an attribute. The SMF/section_comments fixer inserts
+ // the banner between the attribute and its method.
+ #[DataProvider('maxTopicSizeProvider')]
+ public function testGetPostsPerPage(mixed $enable_all_messages, int $expected): void
+ {
+ Config::$modSettings['enableAllMessages'] = $enable_all_messages;
+
+ $this->assertSame($expected, $this->getPostsPerPage());
+ }
+
+ /***********************
+ * Public static methods
+ ***********************/
+
+ /**
+ * @return array
+ */
+ public static function maxTopicSizeProvider(): array
+ {
+ return [
+ 'the admin sets the size' => ['50', 50],
+ 'an integer is accepted too' => [50, 50],
+ 'zero means the admin never shows all' => ['0', TopicPrint::DEFAULT_MAX_POSTS],
+ 'a negative size is meaningless' => ['-50', TopicPrint::DEFAULT_MAX_POSTS],
+ 'so is a non-numeric one' => ['lots', TopicPrint::DEFAULT_MAX_POSTS],
+ ];
+ }
+
+ /******************
+ * Internal methods
+ ******************/
+
+ protected function tearDown(): void
+ {
+ unset(Config::$modSettings['enableAllMessages']);
+ }
+
+ /**
+ * Calls the protected helper under test.
+ */
+ private function getPostsPerPage(): int
+ {
+ $action = (new \ReflectionClass(TopicPrint::class))->newInstanceWithoutConstructor();
+
+ return (new \ReflectionMethod(TopicPrint::class, 'getPostsPerPage'))->invoke($action);
+ }
+}