Minor - format code

certificates/index.php

@@ -42,7 +42,13 @@ switch ($action) {
             $pdfName = api_replace_dangerous_char(get_lang('Certificate') . ' ' . $userInfo['username']);
 
             $pdf = new PDF($pageFormat, $pdfParams['orientation'], $pdfParams);
-            $pdf->html_to_pdf($certificatePathList, $pdfName, null, false, false);
+            $pdf->html_to_pdf(
+                $certificatePathList,
+                $pdfName,
+                null,
+                false,
+                false
+            );
         }
         break;
     default:

main/inc/lib/pdf.lib.php

@@ -177,9 +177,12 @@ class PDF
      *     0 => array('title'=>'Hello','path'=>'file.html'),
      *     1 => array('title'=>'Bye','path'=>'file2.html')
      * );
-     * @param   string  pdf name
-     * @param   string  course code (if you are using html that are located in the document tool you must provide this)
-     * @param bool Whether to print the header, footer and watermark (true) or just the content (false)
+     * @param string $pdf_name pdf name
+     * @param string $course_code (if you are using html that are located in the document tool you must provide this)
+     * @param bool $print_title add title
+     * @param bool $complete_style show header and footer if true
+     * @param bool $addStyle
+     *
      * @return bool
      */
     public function html_to_pdf(
@@ -190,10 +193,6 @@ class PDF
         $complete_style = true,
         $addStyle = true
     ) {
-        if ($complete_style === false) {
-            error_log(__FUNCTION__.' with no style');
-        }
-
         if (empty($html_file_array)) {
             return false;
         }
@@ -224,6 +223,7 @@ class PDF
 
         // Formatting the pdf
         self::format_pdf($course_data, $complete_style);
+
         $counter = 1;
         foreach ($html_file_array as $file) {
             //Add a page break per file
@@ -261,7 +261,6 @@ class PDF
             if ($addStyle) {
                 $css_file = api_get_path(SYS_CSS_PATH).'/print.css';
                 $css = file_exists($css_file) ? @file_get_contents($css_file) : '';
-
                 $this->pdf->WriteHTML($css, 1);
             }
 
@@ -282,7 +281,7 @@ class PDF
 
                 if ($extension === 'html') {
                     $filename = basename($filename, '.html');
-                } elseif($extension === 'htm'){
+                } elseif ($extension === 'htm') {
                     $filename = basename($filename, '.htm');
                 }
 
@@ -328,7 +327,6 @@ class PDF
                                             if (strpos($old_src, 'courses/'.$course_data['path'].'/document/') !== false) {
                                                 $old_src_fixed = str_replace('courses/'.$course_data['path'].'/document/', '', $old_src);
                                             } else {
-
                                                 // Try with the dirname if exists
                                                 if (file_exists($dirName.'/'.$old_src)) {
                                                     $document_path = '';
@@ -838,7 +836,10 @@ class PDF
                     $watermark_text = api_get_setting('pdf_export_watermark_text');
                 }
                 if (!empty($watermark_text)) {
-                    $this->pdf->SetWatermarkText(strcode2utf($watermark_text),0.1);
+                    $this->pdf->SetWatermarkText(
+                        strcode2utf($watermark_text),
+                        0.1
+                    );
                     $this->pdf->showWatermarkText = true;
                 }
             }
@@ -846,8 +847,8 @@ class PDF
             if (empty($this->custom_header)) {
                 self::set_header($course_data);
             } else {
-                $this->pdf->SetHTMLHeader($this->custom_header,'E');
-                $this->pdf->SetHTMLHeader($this->custom_header,'O');
+                $this->pdf->SetHTMLHeader($this->custom_header, 'E');
+                $this->pdf->SetHTMLHeader($this->custom_header, 'O');
             }
 
             if (empty($this->custom_footer)) {