Menu

#313 Payment Selection for Payroll integrate with the AD Core

open-rejected
5
2010-06-14
2010-06-14
No

Payment Selection for Payroll integrate with the ADempiere Core.

So as in the payment selection for AP invoice we need the same functionality to imprint the print cheques from all employes

Kind regards
Victor Perez
www.e-evolution.com

Discussion

  • Waqar Aziz

    Waqar Aziz - 2010-06-14

    there's some problem in the sql script file
    "\migration\354a-trunk\postgresql736_FR_3015702_HR_PaySelection"

    when I tried to execute it in PostgreSQL's pgAdmin 3
    it gave me the following error

    ERROR: syntax error at end of input
    LINE 3088: ...TAMP('2009-12-02 18:43:09','YYYY-MM-DD HH24:MI:SS'),UpdatedB

    when I checked it out, I came up to know that the last line of this script file

    "UPDATE AD_Column SET Description='Payroll Payment Selection', Help='The Payroll Payment Selection identifies a unique Payment', IsUpdateable='N',Updated=TO_TIMESTAMP('2009-12-02 18:43:09','YYYY-MM-DD HH24:MI:SS'),UpdatedB"

    is having some uncompleted line of syntax

     
  • Carlos Ruiz

    Carlos Ruiz - 2010-06-14
    • status: open --> open-rejected
     
  • Carlos Ruiz

    Carlos Ruiz - 2010-06-14

    Reverted these commits with revision 12570.

    I'm sorry to revert this way but I'm in the middle of a release, and I'm
    trying to release the most clean environment.

    Reasons to revert:

    In the peer review I'm conducting I see wrong convention in names of tables and views, or probably a wrong commit creating a table

    I'm in doubt if C_HR_PaySelection_Check_v is a view, is a table?
    Why the C_ prefix?
    Why there is not C_HR_PaySelection_Check_VT ?

    Suggestion:
    Please commit complete and working things - following the naming conventions inherited.

    Regards,

    Carlos Ruiz

     

Log in to post a comment.