gamonoid / icehrm

Manage your employees easily with a robust and efficient Human Resource Management System
http://icehrm.com
Other
579 stars 373 forks source link

Payroll Columns add new not Working V27.0.1 - v27.0.2 and mobile App and more #219

Open mbtaweel opened 4 years ago

mbtaweel commented 4 years ago

This platform is very beautiful I tried to install from the latest update 1- There is a problem with Payroll Columns Found in version 20 it works fine Also in version 26 it works well 2- Also the mobile application is not working in all versions 3- I want to explore the features until I decide to move to the paid plan or change the platform Please are there updates soon to fix these problems 4- Are there plans to include advanced benefits in employment such as import by uploading CVs in one group? 5- Can payslip template be made as table 6- Importing data Downloading templates does not work

7 - Is it possible to update from an old version to a new version and what is the method? thank you very much

ravinamdev1 commented 3 years ago

Hi Team,

I am also facing the same problem with Version 27.0.2.OS Release Date: 10/06/2020. Request you to please provide the solution.

Thanks in Advance.

julyprum commented 3 years ago

I am also having this issue with latest version (27.0.2.OS). When trying to create Payroll Columns, the Add New button does not responds.

Chrome console displays:

1 Uncaught TypeError: n.replace is not a function at n.value (payroll.js?v=27.0.2.OS:1) at n.value (payroll.js?v=27.0.2.OS:1) at HTMLButtonElement.onclick (?g=admin&n=payroll&m=admin_Payroll:1)

julyprum commented 3 years ago

ok, was able to "fix it".

look for payroll.js inside web\admin\dist

then pretty print it, around line 3050 you will find: "datagroup" === e[1].type || "signature" === e[1].type || "tinymce" === e[1].type || e[1].type,

right below that, add this: n = ""+n;

The issue seems to be caused by the field templates for the Payroll Columns, for the "code" column is sending false instead of the HTML template for the field, hence failing the n.replace