css import uses wrong path

binford2k - September 22, 2006 - 19:49
Project:Tables Filter Module
Version:4.7.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

The string to use when getting module paths is 'module' rather than 'modules'

root@lug:~/drupal/modules# diff tables/tables.module contrib/tables/tables.module  -u
--- tables/tables.module        2006-09-03 02:06:17.000000000 -0700
+++ contrib/tables/tables.module        2006-09-22 12:49:20.000000000 -0700
@@ -180,7 +180,7 @@

function tables_menu($may_cache) {
   if (!$may_cache) {
-    if ($css = variable_get('tables_css', drupal_get_path('modules','tables').'/tables.css')) {
+    if ($css = variable_get('tables_css', drupal_get_path('module','tables').'/tables.css')) {
       theme('add_style', $css);
     }
   }
@@ -188,12 +188,12 @@
}

function tables_settings() {
-  if (!file_check_location(variable_get('tables_css', drupal_get_path('modules','tables').'/tables.css'))) {
+  if (!file_check_location(variable_get('tables_css', drupal_get_path('module','tables').'/tables.css'))) {
     $error['tables_css'] = theme('error', t('File does not exist, or is not readable.'));
   }
   $form["tables_css"] = array('#type' => 'textfield',
                              '#title' => t('Style Sheet'),
-                             '#default_value' => variable_get('tables_css', drupal_get_path('modules','tables').'/tables.css'),
+                             '#default_value' => variable_get('tables_css', drupal_get_path('module','tables').'/tables.css'),
                              '#size' => 70,
                              '#maxlength' => 255,
                              '#description' => t("Specify the relative path to your tables style sheet.  The style sheet specified here will be used to style tables you insert.  If you prefer to style your tables in your theme, you can leave this field blank. ". $error['tables_css']),

#1

webgeer - September 10, 2009 - 06:23
Status:reviewed & tested by the community» fixed

#2

System Message - September 24, 2009 - 06:30
Status:fixed» closed

Automatically closed -- issue fixed for 2 weeks with no activity.

 
 

Drupal is a registered trademark of Dries Buytaert.