That's a good point Jacopo. It seems like in a lot of places the account code has more meaning than the account name since the account code often implies the accounts place in the hierarchy of accounts, there can be subtle differences between accounts that are not well represented in the names, and just like we do with things like Jira issue numbers once you refer to it a few times it is easier to remember the code than it is to remember the name.

I guess the most important point for a drop-down is seeing the accounts in their normal order, with hierarchy and all, which is what you get when sorted by the accountCode.

Anyway, +1 for changing it back... or perhaps not exactly back but to something like:

${accountCode}-${accountName} [${glAccountId}]

and remove both entity-order-by elements that are there now, and add one for "accountCode". The reason for this is that currently glAccountId is always the same as accountCode for the OOTB GlAccounts, but that may not always be the case for people and in fact the point of having an accountCode that is not a primary key is so that it can be changed over time.

-David


On Jun 28, 2009, at 8:51 AM, Jacopo Cappellato wrote:

Are we sure that this is the best way to go? Usually chart of accounts are prepared using special conventions for account id (i.e. "accountCode"); see for example how the demo data is organized. I think it is important to keep that order; also, most of the time, accountants already know the account codes and use them instead of the descriptions.

Thoughts?

Jacopo

On Jun 28, 2009, at 2:37 AM, [email protected] wrote:

Author: ashish
Date: Sun Jun 28 08:37:58 2009
New Revision: 789074

URL: http://svn.apache.org/viewvc?rev=789074&view=rev
Log:
Trying to make Accountant life easy.
Its easy to find general ledger from alphabet instead of number.

Modified:
ofbiz/trunk/applications/accounting/widget/GlobalGlAccountsForms.xml

Modified: ofbiz/trunk/applications/accounting/widget/ GlobalGlAccountsForms.xml
URL: 
http://svn.apache.org/viewvc/ofbiz/trunk/applications/accounting/widget/GlobalGlAccountsForms.xml?rev=789074&r1=789073&r2=789074&view=diff
= = = = = = = = = ===================================================================== --- ofbiz/trunk/applications/accounting/widget/ GlobalGlAccountsForms.xml (original) +++ ofbiz/trunk/applications/accounting/widget/ GlobalGlAccountsForms.xml Sun Jun 28 08:37:58 2009
@@ -132,7 +132,8 @@
     </field>
     <field name="parentGlAccountId">
         <drop-down allow-empty="true">
- <entity-options key-field-name="glAccountId" entity-name="GlAccount" description="[${glAccountId}] $ {accountName}-${accountCode}"> + <entity-options key-field-name="glAccountId" entity-name="GlAccount" description="${accountName} [$ {glAccountId}]-${accountCode}">
+                    <entity-order-by field-name="accountName"/>
                 <entity-order-by field-name="glAccountId"/>
             </entity-options>
         </drop-down>




Reply via email to