HTML entities possibly cut off when searching for a string

Ok, after having had this in the back of my head for some time, I thought that the simple fix would be to cut the string first, and apply htmlspecialchars then (since this function removes any entities, tags etc which would - when cut off - generate invalid markup, the problems is solved.)

To do this, change lines 1053/1054 of includes/locale.inc, which look like this:

      $source = htmlspecialchars($value['source']);
      $rows[] = array(array('data' => (strlen($source) > 150 ? substr($source, 0, 150) .'...' : $source) .'<br /><small>'. $value['location'] .'</small>'), array('data' => _locale_string_language_list($value['locales']), 'align' => 'center'), array('data' => l(t('edit'), "admin/locale/string/edit/$lid"), 'nowrap' => 'nowrap'), array('data' => l(t('delete'), "admin/locale/string/delete/$lid"), 'nowrap' => 'nowrap'));

to this:

      $source = htmlspecialchars(strlen($value['source']) > 150 ? substr($value['source'], 0, 150) .'...' : $value['source']);
      $rows[] = array(array('data' => $source .'<br /><small>'. $value['location'] .'</small>'), array('data' => _locale_string_language_list($value['locales']), 'align' => 'center'), array('data' => l(t('edit'), "admin/locale/string/edit/$lid"), 'nowrap' => 'nowrap'), array('data' => l(t('delete'), "admin/locale/string/delete/$lid"), 'nowrap' => 'nowrap'));

(the line numbers are valid for my Drupal 4.5.1.)

Regards,
Milan