Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.
Even though the above link is broken, I was able to download the patch using wget http://drupal.org/files/issues/t.uc_order.module.patch.
After reviewing, I see that almost all of the changes involve wrapping t() around the 'title' and 'description' values defined in hook_menu(). This is a wrong use of t(), as neither of these is supposed to be wrapped in t() in Drupal 6. The remainder of the changes are EALL fixes, which are an entirely different issue than the stated topic and which is properly the subject of another issue http://drupal.org/node/507238
Hence, I'm marking the t() issue as "by design" since the absence of t() on those fields is correct.
Comments
Comment #1
Island Usurper commentedAttachment is broken.
Comment #2
tr commentedEven though the above link is broken, I was able to download the patch using
wget http://drupal.org/files/issues/t.uc_order.module.patch.After reviewing, I see that almost all of the changes involve wrapping t() around the 'title' and 'description' values defined in hook_menu(). This is a wrong use of t(), as neither of these is supposed to be wrapped in t() in Drupal 6. The remainder of the changes are EALL fixes, which are an entirely different issue than the stated topic and which is properly the subject of another issue http://drupal.org/node/507238
Hence, I'm marking the t() issue as "by design" since the absence of t() on those fields is correct.