Docs: Fix placement of some duplicate hook references.

Hook documentation should be on the line directly above the line containing the `do_action()` or `apply_filters()` call. The CS auto-fixing, which changed some inconsistent function calls to multi-line function calls, is part of the reason why this was no longer the case for a select group of duplicate hook references.

Includes minor code layout fixes.

See #47110.

git-svn-id: https://develop.svn.wordpress.org/trunk@46088 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
Sergey Biryukov
2019-09-10 19:22:07 +00:00
parent b3b64db570
commit 159c78c853
15 changed files with 329 additions and 216 deletions
+20 -20
View File
@@ -16,11 +16,11 @@ class Tests_Paginate_Links extends WP_UnitTestCase {
$page50 = get_pagenum_link( 50 );
$expected = <<<EXPECTED
<span aria-current='page' class='page-numbers current'>1</span>
<a class='page-numbers' href='$page2'>2</a>
<a class='page-numbers' href='$page3'>3</a>
<span aria-current="page" class="page-numbers current">1</span>
<a class="page-numbers" href="$page2">2</a>
<a class="page-numbers" href="$page3">3</a>
<span class="page-numbers dots">&hellip;</span>
<a class='page-numbers' href='$page50'>50</a>
<a class="page-numbers" href="$page50">50</a>
<a class="next page-numbers" href="$page2">Next &raquo;</a>
EXPECTED;
@@ -34,11 +34,11 @@ EXPECTED;
$page50 = home_url( '/page/50/' );
$expected = <<<EXPECTED
<span aria-current='page' class='page-numbers current'>1</span>
<a class='page-numbers' href='$page2'>2</a>
<a class='page-numbers' href='$page3'>3</a>
<span aria-current="page" class="page-numbers current">1</span>
<a class="page-numbers" href="$page2">2</a>
<a class="page-numbers" href="$page3">3</a>
<span class="page-numbers dots">&hellip;</span>
<a class='page-numbers' href='$page50'>50</a>
<a class="page-numbers" href="$page50">50</a>
<a class="next page-numbers" href="$page2">Next &raquo;</a>
EXPECTED;
@@ -58,12 +58,12 @@ EXPECTED;
$page50 = get_pagenum_link( 50 );
$expected = <<<EXPECTED
<a class='page-numbers' href='$home'>1</a>
<span aria-current='page' class='page-numbers current'>2</span>
<a class='page-numbers' href='$page3'>3</a>
<a class='page-numbers' href='$page4'>4</a>
<a class="page-numbers" href="$home">1</a>
<span aria-current="page" class="page-numbers current">2</span>
<a class="page-numbers" href="$page3">3</a>
<a class="page-numbers" href="$page4">4</a>
<span class="page-numbers dots">&hellip;</span>
<a class='page-numbers' href='$page50'>50</a>
<a class="page-numbers" href="$page50">50</a>
EXPECTED;
$links = paginate_links(
@@ -84,12 +84,12 @@ EXPECTED;
$expected = <<<EXPECTED
<a class="prev page-numbers" href="$home">&laquo; Previous</a>
<a class='page-numbers' href='$home'>1</a>
<span aria-current='page' class='page-numbers current'>2</span>
<a class='page-numbers' href='$page3'>3</a>
<a class='page-numbers' href='$page4'>4</a>
<a class="page-numbers" href="$home">1</a>
<span aria-current="page" class="page-numbers current">2</span>
<a class="page-numbers" href="$page3">3</a>
<a class="page-numbers" href="$page4">4</a>
<span class="page-numbers dots">&hellip;</span>
<a class='page-numbers' href='$page50'>50</a>
<a class="page-numbers" href="$page50">50</a>
<a class="next page-numbers" href="$page3">Next &raquo;</a>
EXPECTED;
@@ -333,7 +333,7 @@ EXPECTED;
)
);
$this->assertContains( "<span aria-current='page' class='page-numbers current'>3</span>", $links );
$this->assertContains( '<span aria-current="page" class="page-numbers current">3</span>', $links );
}
/**
@@ -355,6 +355,6 @@ EXPECTED;
);
$page_2_url = home_url() . '?foo=2';
$this->assertContains( "<a class='page-numbers' href='$page_2_url'>2</a>", $links );
$this->assertContains( "<a class=\"page-numbers\" href=\"$page_2_url\">2</a>", $links );
}
}