[Erp5-dev] [PATCH] erp5_accounting_l10n_pl small patch for Organisation_viewDetails

[Jean-Paul Smets]

Le mercredi 22 novembre 2006 00:04, bartek a écrit :
> Jérôme Perrin wrote:
> > Le Mardi 21 Novembre 2006 13:55, Łukasz Nowak a écrit :
> >> Hello,
> >>
> >>  - used searchFolder instead of object values
> >>  - removed id (to follow guidelines)
> >>  - cleaned up Folder_viewContentList invocation [*]
> >
> > Hello
> > why do we need a custom Organisation_viewDetails for l10n_pl ?
>
> Because to be able to properly fill tax forms we have to define to which
> tax office a given organisation reports, and I placed this control on
> Organisation_viewDetails form.
>
> Now that you mention it, it'd be probably better to place
> country-specific controls on a separate form to prevent conflicts (which
> is probably what you just had in mind...).

I do not like to put hard coded stuff in organisation. We need a generic 
mechanism to associate organisation (ex. tax office) to organisations (ex. 
branch office). So, I will probably request current implementation to be 
removed.

The clean way is to associate the tax office in the accounting transaction 
line. This way, it is possible to calculate the amoun of due tax per tax 
office. And it even possible for an organisation to change its tax office 
during the year or to have multiple tax offices.

Filling this property automatically can simply be a side effect of 
preference selection.

JPS.

>
> Bartek
>
> >> I have question for [*] - invocation of Folder_viewContentList. If for
> >> eg. listbox is restricting its view only to some portal_types shall be
> >> Folder_viewContentList restricted too, or shall it show all
> >> portal_types in root object?
> >
> > When you have a listbox in view mode, clicking on listbox's title
> > should bring you in list mode on a listbox showing the *same lines*,
> > where you can use list mode features (cut, copy, paste, delete etc) on
> > the same dataset.
> >
> > Today, only a few forms respect this. I don't know what others think,
> > but maybe we could set a TALES expression for list action in
> > Organisation_viewDetails/listbox, something like :
> > python:'list?' + modules['ZTUtils'].make_query(portal_type=[x[0] for x
> > in field.get_value('portal_types')])
> >
> > Jérome
> > _______________________________________________
> > Erp5-dev mailing list
> > Erp5-dev at erp5.org
> > http://erp5.org/mailman/listinfo/erp5-dev
>
> _______________________________________________
> Erp5-dev mailing list
> Erp5-dev at erp5.org
> http://erp5.org/mailman/listinfo/erp5-dev

-- 
Jean-Paul Smets-Solanes, Nexedi CEO - Tel. +33(0)6 62 05 76 14
Nexedi: Consulting and Development of Libre / Open Source Software
http://www.nexedi.com
ERP5: Libre/ Open Source ERP Software for small and medium companies
http://www.erp5.org
Rentalinux: Desktop Linux Server
http://www.rentalinux.com