This is a minor side issue which arose at the tail end of #1816764: Add missing type hinting to Contextual module docblocks
Normally with these bug/issues the documentation no longer described the actual implementation.
This one is in reverse...
here is a code cut and past from https://www.drupal.org/node/1816764#comment-9703923
The functions current returns null.
+++ b/core/modules/contextual/src/Tests/ContextualDynamicContextTest.php
@@ -134,8 +137,6 @@ function testDifferentPermissions() {
- *
- * @return bool
*/
protected function assertContextualLinkPlaceHolder($id) {
@@ -146,8 +147,6 @@ protected function assertContextualLinkPlaceHolder($id) {
- *
- * @return bool
*/
protected function assertNoContextualLinkPlaceHolder($id) {
Beta phase evaluation
Comments
Comment #1
Anonymous (not verified) commentedAdded the returns and fixed the comments.
Also added a beta eval to the summary.
Edit: I only just noticed that you assigned this issue to yourself while making it. I'm sorry, hope you didn't start fixing this yet :/
Comment #2
martin107 commentedAh no problem, I was interrupted.
Its looks good to me. I won't RTBC my own issue... but try and trade reviews with someone in IRC tomorrow.
Comment #3
geertvd commentedLooks like the sane thing to do.
Comment #4
alexpottTest only change is permitted in beta. Committed 7480281 and pushed to 8.0.x. Thanks!