#1057 General Journal -> Chart of Accounts, select account broken

1.4
closed-fixed
Chris Travers
None
5
2014-02-14
2014-02-09
John Locke
No

In earlier versions of 1.4, clicking an account on the COA list would take you to a GL report screen with that account selected.

Now you see the account flash in that box, and then get blanked out. Dojo hookup issue.

Discussion

  • John Locke
    John Locke
    2014-02-09

    Would also like to know if there is a replacement "register" report that keeps a running account balance corresponding to a single account -- this seems to be missing in 1.4, unless I'm overlooking something?

     
  • Chris Travers
    Chris Travers
    2014-02-10

    • assigned_to: Chris Travers
     
  • Chris Travers
    Chris Travers
    2014-02-10

    John, In 1.4, the register report shows extra admin columns if the user has permission to administrate it.

     
  • Chris Travers
    Chris Travers
    2014-02-10

    ok, so I committed svn rev 6835 to try to solve this problem but no luck.

    I think what is happening is that the value is set before the store initializes. Is there an easy way around this? At any rate I am not going to say this is blocking for beta 5. When you get a chance would like to discuss how to address this on IRC though.

     
  • Chris Travers
    Chris Travers
    2014-02-11

    • status: open --> closed-fixed
     
  • Chris Travers
    Chris Travers
    2014-02-11

    svn 6838

     
  • adapted

    UI/accounts/AccountStore.js
    to emit accountstore_loadcomplete event

    UI/accounts/AccountSelector.js
    to save initialValue
    listen on accountstore_loadcomplete
    to reset initialValue

    UI/Reports/filters/gl.html
    for test , reset previous setup

    Committed revision 6856.

     
  • John Locke
    John Locke
    2014-02-14

    Hah, just now realized tshvr == Herman... ;-)

    Been swamped, but those fixes look like the approach I was thinking -- capture the value before instantiating the widget, then passing the value into the constructor (or setting later if necessary).

    One thing I did see in the most recent commits: console.log at least used to break IE -- need to make sure those get commented out/removed before production.