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

Topic: Remove redundant line in reporting/includes/header2.inc

Line in reporting/includes/header2.inc:

        $this->LineTo($right, $iline2 ,$right, $iline4);

is redundant and can be removed as it is already executed in the preceding loop.

@joe: please do the needful.

apmuthu's Website

  • joe
  • Administrator
  • Offline
  • Registered: 03/15/2007
  • Posts: 5,085

Re: Remove redundant line in reporting/includes/header2.inc

Yes, but only for 2.4. This is not a big deal so we only do it for 2.4.

Unstable repo updated.

/Joe

3 Reply by apmuthu (edited by apmuthu 08/01/2016 07:09:51 am)

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

Re: Remove redundant line in reporting/includes/header2.inc

So my FA 2.3 GitHub FAMods folder grows.... Soon it will be the defacto v2.3 of choice due to lack of backports into 2.3's official repo for whatever reason - for those who wish to remain at 2.3.

apmuthu's Website