As reported by xjm, the readmore link is not rendered anymore.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner’s picture

Status: Active » Needs review
FileSize
4.5 KB

Let's see whether the tests are working.

Status: Needs review » Needs work
Issue tags: -VDC

The last submitted patch, drupal-1970136-1.patch, failed testing.

oresh’s picture

Status: Needs work » Needs review
Issue tags: +VDC

#1: drupal-1970136-1.patch queued for re-testing.

oresh’s picture

Status: Needs review » Reviewed & tested by the community

Before applying the patch the 'More link' in pager section:

PAGER
Use pager:Full | Paged, 10 items
More link:Yes

Was set to Yes, but no link on the bottom of the view did not appear.

After applying the patch if showed me the link with correct text and link.
The "Display 'more' link only if there is more content" also works correctly. When enabled, and displaying 1 item on a page, i get it everywhere except the last page (as intended).
Doesn't get duplicated, markup is ok. So RTBC.
Thaks.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, drupal-1970136-1.patch, failed testing.

dawehner’s picture

Forgot to add the config file.

PHP is really odd:

      
<div class="more-link">
  <a href="/d8/test_display_more">
    custom more text  </a>
</div>

This html is read by simplexml into:

array (
  0 => 
  SimpleXMLElement::__set_state(array(
     '@attributes' => 
    array (
      'href' => '/d8/test_display_more',
    ),
  )),
)
joelpittet’s picture

#1 seems to fix the problem though I am not about the testbot failure, nor the intention behind converting some of the tests to private test functions...

thedavidmeister’s picture

the testbot failed because of a fatal error in a test introduced by the patch.

dawehner’s picture

Status: Needs work » Needs review
Issue tags: -VDC

#1: drupal-1970136-1.patch queued for re-testing.

Status: Needs review » Needs work
Issue tags: +VDC

The last submitted patch, drupal-1970136-1.patch, failed testing.

joelpittet’s picture

@dawehner #6 references you are missing a yml file for 'test_display_more' and left and not sure if the test is fully finished with the private function changes I think were done for testing?

dawehner’s picture

Status: Needs work » Needs review
FileSize
5.91 KB

Right I forgot to add the file, though I have no idea why this xpath doesn't return the actual link title.
Maybe someone can help.

Status: Needs review » Needs work

The last submitted patch, drupal-1970136-12.patch, failed testing.

olli’s picture

That xpath looks good to me, but there is some spaces from views-more.tpl.php. Added trim().

olli’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work
Issue tags: -VDC

The last submitted patch, drupal-1970136-14.patch, failed testing.

olli’s picture

Status: Needs work » Needs review
Issue tags: +VDC

#14: drupal-1970136-14.patch queued for re-testing.

joelpittet’s picture

Status: Needs review » Reviewed & tested by the community

Great work guys!

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 91e164e and pushed to 8.x. Thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.