Decode HTML entities in email subject.

[RoSk0]

Fix for the subject issue as well as small code style improvements.

Drupal.org issue page https://www.drupal.org/node/2386273.

[amitaibu]

I see in the original issue:

In building the subject line, the MessageNotifierEmail class strips tags from the text but it fails to decode HTML entities. So a subject line which should be 'New post on the topic "some title"' actually comes out as 'New post on the topic "some title"'.

I can't see the difference in that above example ;) Also, it would be great if it had a simpleTest -- can you write one?

[RoSk0]

Fixed issue description.
I'm not really experienced with SimpleTest so I have failed to write a test, but what I have came up with is this:

  $test_str = "Sting ' with " . 'quotes " and HTML entities &.';
  $msg_type = 'test_email';
  $message_type_obj = message_type_create($msg_type);
  $wrapper = entity_metadata_wrapper('message_type', $message_type_obj);
  $wrapper->{MESSAGE_FIELD_MESSAGE_TEXT}[] = array('value' => $test_str, 'format' => 'plain_text');
  $wrapper->save();

  $message = message_create($msg_type);
  message_notify_send_message($message, array('mail' => 'test-email@test.org'));
  $output = $message->output;

  $this->assertEqual($output['message_notify_email_subject'], $test_str, 'Entities has been successfully decoded in email subject.');

But my issue here is that I must test a real MessageNotifierEmail class and I'm not sure how exactly I should do this:

1. We do not want it to sent out an emails.
2. We want it to behave exactly like MessageNotifierTest::deliver in terms of storing prepared output in $message object and returning status set on $message.

I want to get familiar with this so any help would be appreciated!

[RoSk0]

Just to be explicit: without this PR with the subject set to >Sting ' with quotes " and HTML entities &.< you will end up receiving email with subject >Sting &#39; with quotes &quot; and HTML entities &amp;.<

[amitaibu]

@ordavidil can you give a shot at the tests please :)

[ordavidil]

@amitaibu
Sure, I'll check it out.