On HelpfulnessBlock's validateForm():
if (!form_state->getValue('helpfulness_rating')) {
$form_state->setErrorByName('helpfulness_rating', $this->t('Please indicate if this page is helpful or not.'));
}
Since when the rating is "No" the value is 0, it would fall on the error condition when it shouldn't. We should be explicit and check for a NULL value here instead.
Comments
Comment #4
gueguerreiroCommitted 37f0652 and pushed to 2.x. Thanks!
Committed 34e51fd and pushed to 8.x-1.x. Thanks!