Hi!

Congrats on the module. He is excellent.

I added function t() for translate message and fixed variable unused.

Thanks.

Regards.

Patch with fix: personal_digest-translatevariable-2833778-2.patch

Img

Img

Img

Comments

renatog created an issue. See original summary.

renatog’s picture

Assigned: renatog » Unassigned
Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new1.4 KB

Patch submitted with the fixes.

Thanks.

Regards.

cebasqueira’s picture

Status: Needs review » Reviewed & tested by the community

Code looks googd... RTBC

gnuget’s picture

Status: Reviewed & tested by the community » Needs work

Hi!

Thanks for your patch.

One small nitpick:

-    drupal_set_message('Check your mail');
+    drupal_set_message(t('Check your mail'));

Here actually you can use $this->t instead the global function.

Regards.

matslats’s picture

Status: Needs work » Fixed
matslats’s picture

Status: Fixed » Closed (fixed)
renatog’s picture

Ok thanks guys @gnuget and @matslats.

Hugs.