Skip to content
Snippets Groups Projects
Commit 0d6028e9 authored by Doug Green's avatar Doug Green
Browse files

#109386 from webchick - add link to warning about info files;

parent 8a94da07
No related branches found
No related tags found
No related merge requests found
......@@ -598,9 +598,9 @@ function _coder_error(&$results, $rule, $lineno = -1, $line = '', $original = ''
$warning = $rule['#warning_callback']();
}
else { // if this happens, there is an error in the rule definition
$warning = t('please !report this !warning',
$warning = t('please <a href="@report">report</a> this !warning',
array(
'!report' => l('report', 'http://drupal.org/node/add/project_issue/coder/bug'),
'@report' => 'http://drupal.org/node/add/project_issue/coder/bug',
'!warning' => $rule['#warning_callback'],
)
);
......
......@@ -68,7 +68,7 @@ function _coder_50_callback(&$coder_args, $rule, $lines, &$results) {
// make sure that a .info file exists
$filename = drupal_substr(realpath($filename), 0, -7) .'.info';
if (!file_exists($filename)) {
$results[0] = theme('coder_warning', $lineno, t('All modules now need to have a modulename.info file'));
$results[0] = theme('coder_warning', $lineno, t('All modules now need to have a <a href="@info">modulename.info file</a>', array('@info' => 'http://drupal.org/node/101009')));
}
}
}
......
......@@ -188,9 +188,9 @@ function _coder_style_core_global_regex() {
*/
function _coder_style_xhtml_warning() {
return t('use lowercase html tags to comply with !XHTML',
return t('use lowercase html tags to comply with <a href="@XHTML">XHTML</a>',
array(
'!XHTML' => l('XHTML', 'http://www.w3.org/TR/xhtml1/#h-4.2'),
'@XHTML' => 'http://www.w3.org/TR/xhtml1/#h-4.2',
)
);
}
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment