Fix #771300: warn the site admin if dompdf's lib/fonts is not writable
authorJoao Ventura
Fri, 16 Jul 2010 20:23:37 +0000 (20:23 +0000)
committerJoao Ventura
Fri, 16 Jul 2010 20:23:37 +0000 (20:23 +0000)
INSTALL.txt
README.txt
print_pdf/print_pdf.module

index 6d5e512..afe9ea4 100644 (file)
@@ -5,8 +5,7 @@ Decompress the print-n.x-n.n.tar.gz file into your Drupal modules
 directory (usually sites/all/modules, see http://drupal.org/node/176044 for
 more information).
 
-Enable the print module: Administer > Site building > Modules
-  (admin/build/modules)
+Enable the print module: Administer > Modules (admin/modules)
 
 PDF TOOL
 --------
index cbc1a85..1ed90c3 100644 (file)
@@ -24,7 +24,7 @@ CONFIGURATION
 
 - There are several settings that can be configured in the following places:
 
-  Administer > Site building > Modules (admin/build/modules)
+  Administer > Modules (admin/modules)
     Enable or disable the module. (default: disabled)
 
   Administer > People > Permissions (admin/people/permissions)
index be38975..6903af0 100644 (file)
@@ -173,6 +173,19 @@ function print_pdf_requirements($phase) {
           'severity' => REQUIREMENT_ERROR,
         );
       }
+      else {
+        if (basename($print_pdf_pdf_tool) == 'dompdf_config.inc.php') {
+          $directory = dirname($print_pdf_pdf_tool) .'/lib/fonts';
+          if (!is_dir($directory) || !is_writable($directory)) {
+            $requirements['print_pdf_tool'] = array(
+              'title' => $t('DOMPDF font cache directory'),
+              'value' => $t('Non-writable permissions'),
+              'description' => $t('You must change the %fontdir permissions to be writable, as dompdf requires write-access to that directory.', array('%fontdir' => $directory)),
+              'severity' => REQUIREMENT_ERROR,
+            );
+          }
+        }
+      }
       break;
   }
   return $requirements;