276

(66 replies, posted in Modules Add-on's)

@apmuthu

It's a reference to payslip_details table for querying employees transactions details, I know this is not good solution but have not figured out a better way.

277

(14 replies, posted in Report Bugs here)

As you mentioned we are only interested in the produced numbers.
I think the flowing is ok :

$sql .= "AND s.mb_flag='M' AND m.type = ".ST_WORKORDER."  ";

278

(14 replies, posted in Report Bugs here)

And even location adjustments can cause the wrong statistics

279

(14 replies, posted in Report Bugs here)

No @Joe, what about location transfer ?

280

(14 replies, posted in Report Bugs here)

Which transaction can make m.qty < 0 AND m.type = " .ST_ CUSTCREDIT . " ? @apmuthu

seems $sql .= "AND s.mb_flag='M' AND m.qty > 0 AND m.type <> ".ST_CUSTCREDIT." "; is working.

281

(14 replies, posted in Report Bugs here)

I'm not sure this is a bug, Top 10 Manufactured Items in fiscal year just confuses me on it's presents.

Once I create a work order then manufacturing dashboard shows:
_________________________________________________
|     Items      |           Amount         |          Quantity        |
|-------------------------------------------------------------------|
|AP Surf Set    |                    366.00  |                        1.00 |
|-------------------------------------------------------------------|

After item is sold  the table shows

_________________________________________________
|     Items      |           Amount         |          Quantity        |
|-------------------------------------------------------------------|
|AP Surf Set    |                        0.00  |                        0.00 |
|-------------------------------------------------------------------|

1. Does this widget just present on hand manufactured items ?
2. Should it be changed to all manufactured items includes sold item?

If 1 : this does not make sense much
If 2 : Line 275 of dashboard.inc need to change to $sql .= "AND s.mb_flag='M' AND m.qty > 0 ";

does it need to change?

Insert add_access_extensions(); after line 128 of renderer file will fix the problem.

I guess not only this theme has the problem.

Seems @apmuthu's fixs working.

Can you post your renderer.php code? I did not see can_access_page() in line 142 of original dropdown theme renderer.php

At the moment the Default Credit Limit input box in System and General GL Setup is text format, should it be changed to price format ?

admin/gl_setup.php

line 132: $_POST['default_credit_limit'] = $myrow['default_credit_limit'];
              $_POST['default_credit_limit'] = price_format($myrow['default_credit_limit']);

line 180: text_row(_("Default Credit Limit:"), 'default_credit_limit', $_POST['default_credit_limit'], 12, 12);
              amount_row(_("Default Credit Limit:"), 'default_credit_limit', $_POST['default_credit_limit']);

Maybe when the PO file updates strings from source it finds all _() functions and put whatever in it to be translated, may be at that time $help_context is undefined so it will not end up to PO file.

So we should not put a variable to gettext function.
I'm not sure this analyse is right but if change code as follow it will work.

if (isset($_GET['FixedAsset'])) {
    $help_context = _("Fixed Assets Locations");
    $_POST['fixed_asset'] = 1;
} else
    $help_context = _("Inventory Locations");

page($help_context);

And while digging to this problem I detect the other that after delete of a supplier there is no notification shown.

Both suppliers.php and customers.php also need to be fixed too.

290

(18 replies, posted in Report Bugs here)

why both Sourceforge repo and GitHub update PRIMARY KEY (`type`,`trans_no`,`debtor_no`)  for table supp_trans ?
It must be PRIMARY KEY(`type`,`trans_no`,`supplier_id`)

am I wrong ?

If you set $dflt_xr_provider = 2 to get exchange from Google system will show following errors:


Undefined variable: site_ip in file: C:\xampp\htdocs\fa242\gl\includes\db\gl_db_rates.inc at line 178
C:\xampp\htdocs\fa242\gl\includes\db\gl_db_rates.inc:125:    get_extern_rate('EUR','GOOGLE','10/25/2017')
C:\xampp\htdocs\fa242\gl\manage\exchange_rates.php:139:    retrieve_exrate('EUR','10/25/2017')
C:\xampp\htdocs\fa242\gl\manage\exchange_rates.php:218:    display_rate_edit()
number_format() expects parameter 1 to be double, string given in file: C:\xampp\htdocs\fa242\includes\current_user.inc at line 316
C:\xampp\htdocs\fa242\includes\current_user.inc:316:    number_format('','INF','.',',')
C:\xampp\htdocs\fa242\includes\current_user.inc:391:    number_format2('','max')
C:\xampp\htdocs\fa242\gl\manage\exchange_rates.php:139:    maxprec_format('')
C:\xampp\htdocs\fa242\gl\manage\exchange_rates.php:218:    display_rate_edit()

Fixes:

Line 178 of gl_db_rates.inc should be changed to if(isset($site_ip) $site = $site_ip;

Line 152 should be $site = "finance.google.com";

292

(1 replies, posted in Report Bugs here)

at line 404 in gl_bank.php calls function gl_options_controls() which calls div_start('footer') in gl_bank_ui.inc

This is makes 2 elements with same id = 'footer' which would causes conflict.

does this need to change ?

See the function init{} in the frontaccounting.php, here you can disable any module by comment out a corresponding line.

In your case please replace lines 79 - 80 - 81 with the fllowing:

    //$this->add_application(new customers_app());
    //$this->add_application(new suppliers_app());
    //$this->add_application(new inventory_app());

This theme is not an optimal solution for a theme Joe, I simply wrote CSS for all pages one by one so it will not work on new extensions or if there are changes in the core.
If we can figured out a way to restructuring FA to make create themes easier I will be very willing to work on a better one and share it.

First version released with basic functions, looking forward for the feedback to make it better.

296

(13 replies, posted in FA Modifications)

I think we don't need to add a line at the bottom of every group which will make the report confusing.

Just add as follows and don't need the option:
http://imgur.com/ABqVoPe

297

(13 replies, posted in FA Modifications)

Yes, group total is very useful, hoping it will be implemented next release.

another thread here:
https://frontaccounting.com/punbb/viewtopic.php?id=4913

Updated the Demo link in post #1

There is a problem with Person Id  I have faced, when we make transactions with Payable or Receivable account for an Employee or another Person who has the same Id as that of a Supplier or Customer then in the gl trans view always appear like this https://ibb.co/c64ahv

I handle it by adding a prefix to the Employee's id in the gl function to make the Id different from Supplier.

Any Ideas ?

I'm going to start recreate Payroll & HRM module to replace the old one which has been created for practice, from now the updates will be updated here.

DEMO

HOW TO INSTALL?

  1. Download the lastest source code here: https://github.com/notrinos/FrontHrm/ar … ads/FA.zip , extract the downloaded zip file to your FA modules directory then rename the extracted folder to FrontHrm

  2. Go to Setup > Install/Activate Extensions, here you will see two modules with the same name FrontHrm, please install the one that has unknown state. See this Image

  3. After step 2, at the same screen please active the module, see this Image

  4. Go to Setup > Access Setup, select your user role in the top dropdown list, mark all the items in Human Resource section then press Save Role. See this Image

  5. Log out then login again now the module is activated.