Problem/Motivation

Currently the docblock has 

   ...
   * @return bool
   *   TRUE if not found, FALSE if found.
   */

But it should be TRUE if found, FALSE if not found.

Originally reported by cbanman

CommentFileSizeAuthor
#3 3251988-3.patch1.03 KBvijaycs85
#2 3251988-2.patch730 bytesvijaycs85

Comments

vijaycs85 created an issue. See original summary.

vijaycs85’s picture

vijaycs85
Location London, UK
commented
Status: Active » Needs review
StatusFileSize
new 3251988-2.patch730 bytes
vijaycs85’s picture

vijaycs85
Location London, UK
commented
Issue summary: View changes
StatusFileSize
new 3251988-3.patch1.03 KB

Updating the description as well, thanks to @smccabe for the quick offline review.

smccabe’s picture

smccabe commented
Status: Needs review » Reviewed & tested by the community

Confirmed the function does what the new docblock says. RTBC

  • catch committed 640db53 on 10.0.x 
    Issue #3251988 by vijaycs85, smccabe: Update return param of waitForText...

  • catch committed 10d7b48 on 9.4.x 
    Issue #3251988 by vijaycs85, smccabe: Update return param of waitForText...
catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 10.0.x and cherry-picked to 9.4.x, thanks!

Status: Fixed » Closed (fixed)

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