1 Topic by apmuthu (edited by apmuthu 04/20/2013 11:31:44 am)

  • From: Singapore, India, Chennai
  • Registered: 05/02/2009
  • Posts: 6,201

Topic: New rep308.php submitted with updations.

The erstwhile rep_stock_movement extension for FA has Standard Costs as one of the parameters in the SQL and material cost was computed in it. When the extension was merged into the FA core as rep307.php on 2013-01-24, the Standard Material Costs got left out due to paucity of space on the standard A4 and Letter paper sizes and was called Inventory Movement Report.

Another report, rep308.php was made by Janusz/Joe around the same time but was not included in the FA core. Attached herewith is the improved version which is fully functional:

1. Commented out hardcoding of $orientation.
2. Retained UOM (units).
3. Moved extra function below regular function just as in rep307.php
4. Adjusted the indent values of the displayed fields - set for A4 but should work for Letter as well.
5. Ordered the report ids in the reports_main.php

Kindly include it in the FA Core. I have named it Costed Inventory Movement Report for now.

Post's attachments

rep308_files.zip 31.8 kb, 2 downloads since 2013-04-20 

You don't have the permssions to download the attachments of this post.

apmuthu's Website

  • From: Singapore, India, Chennai
  • Registered: 05/02/2009
  • Posts: 6,201

Re: New rep308.php submitted with updations.

Thanks for the updated version now in the FA Core as of HG 3224.

Line 164 in rep308.php:

    $rep->Info($params, $cols, $headers2, $aligns, $cols, $headers, $aligns);

should be:

    $rep->Info($params, $cols, $headers, $aligns, $cols, $headers2, $aligns);

The $headers and $headers2 variables got swapped.

function Info is defined in reporting/includes/pdf_report.inc as

 function Info($params, $cols, $headers, $aligns,
  $cols2 = null, $headers2 = null, $aligns2 = null,
  $companylogoenable = false, $footerenable = false, $footertext = '')

A new string is needed in empty.po:

C&osted Inventory Movement Report

apmuthu's Website

3 Reply by apmuthu (edited by apmuthu 04/21/2013 05:25:30 am)

  • From: Singapore, India, Chennai
  • Registered: 05/02/2009
  • Posts: 6,201

Re: New rep308.php submitted with updations.

It appears that the rep308.php should be the way it is in HG 3224.

This is so because the class variable $headers2 displays first and then the class variable $headers is displayed in the class method FrontReport::Info() in pdf_reports.inc lines 358-367:

        if ($this->headers2 != null)
        {
            $count = count($this->headers2);
            for ($i = 0; $i < $count; $i++)
                $this->TextCol2($i, $i + 1,    $this->headers2[$i]);
            $this->NewLine();
        }
        $count = count($this->headers);
        for ($i = 0; $i < $count; $i++)
            $this->TextCol($i, $i + 1, $this->headers[$i]);

Some involuted logic in the FrontReport class method Info() indeed!

apmuthu's Website