Constructor for NonrenewableTokenException has incorrect message logic

Problem/Motivation

When an access token has expired and can't be renewed, the NonrenewableTokenException thrown has no message. But the constructor has logic that should create a message if one hasn't been passed in. The problem is that it's checking for if(!empty($message) when I think it means if (empty($message)...

Steps to reproduce

Proposed resolution

Change code in constructor that creates a missing message to read:

   if (empty($message) && !empty($grantType)) {
      $message = "A token obtained using the $grantType grant has expired without a refresh, and cannot be renewed without user interaction";
    }

Remaining tasks

User interface changes

API changes

Data model changes