Enable Zoom for Record_ID on HTML Reports

Description

Currently clicking on record id on fact account take to fact account and then need to go to actual transaction window. This is adaxa enhancement contribution for opening window directly from record id click.

Environment

None

Attachments

1

Activity

Show:

Carlos Ruiz February 25, 2015 at 2:56 PM

Thanks , I committed a slightly modified version (littler) - it requires the column AD_Table_ID to be present on the report to enable the zoom on Record_ID - my modifications were mostly to avoid hardcoded references to Fact_Acct

Deepak Pansheriya February 25, 2015 at 1:10 PM

Carlos,
Patch with suggested changes is attached. Please note that no need to use previous patch, So I deleted same. Use this patch only

Carlos Ruiz February 18, 2015 at 6:23 PM

Ah, I thought all the time you were talking about Info Account window - I see now is about reports.

Test case:

I added the column Record_ID to a Fact_Acct report

  • Null pointer exception

  • Adding the Fact_Acct_ID to the report solved the NPE, but doesn't seem correct, I think it must work without adding Fact_Acct_ID, or at least not to throw NPE

BTW - I think this is useful for any table with the columns AD_Table_ID+Record_ID - is it possible to make it wide instead of restricting it just to Fact_Acct?

Regards,

Carlos Ruiz

Deepak Pansheriya February 18, 2015 at 2:54 PM

Attached Patch, Please review and add into dev branch.

Carlos Ruiz February 13, 2015 at 11:38 PM

that sounds better

Fixed

Details

Assignee

Reporter

Labels

Components

Priority

Created February 4, 2015 at 5:14 PM
Updated May 4, 2015 at 5:43 PM
Resolved February 25, 2015 at 2:56 PM