6.0.0-RC7
6/27/26

[#3731] Currencies and Taxes
Summary Currencies and Taxes
Queue Horde Base
Queue Version HEAD
Type Enhancement
State Resolved
Priority 1. Low
Owners
Requester duck (at) obala (dot) net
Created 4/3/06 (7390 days ago)
Due
Updated 5/28/06 (7335 days ago)
Assigned
Resolved 5/28/06 (7335 days ago)
Milestone
Patch No

History
4311 Chuck Hagenbuch Deleted Original Message
 
3511 Chuck Hagenbuch Deleted Original Message
 
2511 Chuck Hagenbuch Deleted Original Message
 
711 Chuck Hagenbuch Deleted Original Message
 
2811 Chuck Hagenbuch Deleted Original Message
 
811 Chuck Hagenbuch Deleted Original Message
 
4711 Chuck Hagenbuch Deleted Original Message
 
4911 Chuck Hagenbuch Comment #14
State ⇒ Resolved
Reply to this comment
Now in the incubator as Horde_Taxes, Horde_Currencies, and the RDO 
stuff is in Horde_RDO for now.
551 duck (at) obala (dot) net Comment #13
New Attachment: modules0506.tgz
Reply to this comment
Added SQL schema and test scripts for api calls.

Now all the initial module is moved to RDO.

Plus the the two template and form extension to browse and edit RDO data.

Now I will wait for an feedback or idea hot to integrate it.


219 duck (at) obala (dot) net Comment #12 Reply to this comment
I don't have such plans. If this is a crucial feature we can do it,
but if it's only 10% of cases or so I think we should consider
leaving it out, at least of the core.
Oky. Lets leave this as is for now.



I think we could adopt a genereal cheking form additional modules for 
every app. When the registry loads the api method list, it could check 
if in the api dir exist a subdir named "modules". If so, it passes the 
subdirs to check if api.php exist in them and loads it. For example 
<api>/modules/taxes/api.php.



Then for every api we can create a submenu entry "modules".



+ horde

++ modules

+++ Taxes -> likend to horde/modules/taxes/taxes.php

+++ Currencies



In this way we can install a module only by copying the files in the 
right place and the additional features will be added to the api list.


199 duck (at) obala (dot) net Comment #11
New Attachment: modules0505.tgz
Reply to this comment
Split code in parts and moved even currencies to RDO.

Added export to XML option to the browsing list, plus reload and 
create actions in the header.

Prepared api calls using RDO.
510 Chuck Hagenbuch Comment #10 Reply to this comment
How about primary keys. I see that currently we support only one
primary key. Are any plans to support combined primary keys?
I don't have such plans. If this is a crucial feature we can do it, 
but if it's only 10% of cases or so I think we should consider leaving 
it out, at least of the core.
295 duck (at) obala (dot) net Comment #9 Reply to this comment
How about having the RDO object track which fields have changed, and
only update those, instead?
This would be nice.



How about primary keys. I see that currently we support only one 
primary key. Are any plans to support combined primary keys?


313 Chuck Hagenbuch Comment #8 Reply to this comment
I found it. The RDO_Mapper::update has the optional second parameter
"fields". But is when instance of RDO is passed as an object this is
ignored. I guess we should check if it is null.
Because you want to update only certain fields?



How about having the RDO object track which fields have changed, and 
only update those, instead?
398 duck (at) obala (dot) net Comment #7 Reply to this comment
I still dont understand RDO::update call good.
Can you say what about it doesn't make sense?
I found it. The RDO_Mapper::update has the optional second parameter 
"fields". But is when instance of RDO is passed as an object this is 
ignored. I guess we should check if it is null.



@ -212,9 +212,7 @@

          if ($object instanceof RDO) {

              $key = $this->describe()->key;

              $id = $object->$key;

-            if (is_null($fields)) {

-                $fields = $object->getFields();

-            }

+            $fields = $object->getFields();

          } else {

              $id = $object;

          }
348 duck (at) obala (dot) net Comment #6
New Attachment: rdoform[3].php
Reply to this comment
Now works all.

This should be cleand up and maybe used for a base of a CRUD forms.
323 Chuck Hagenbuch Comment #5 Reply to this comment
I still dont understand RDO::update call good.
Can you say what about it doesn't make sense?
463 duck (at) obala (dot) net Comment #4
New Attachment: rdoform[2].php
Reply to this comment
Delete and create works.



I still dont understand RDO::update call good.
168 duck (at) obala (dot) net Comment #3
New Attachment: rdoform[1].php
Reply to this comment
- Detect primary key or handle all columns as key like phpMyAdmin

- Delete entry

- Fill up form for editing
597 duck (at) obala (dot) net Comment #2
New Attachment: rdoform.php
Reply to this comment
First try of RDO version of the taxes with form and template autogeneration.
535 duck (at) obala (dot) net Comment #1
Priority ⇒ 1. Low
State ⇒ New
New Attachment: cuta0403.tgz
Queue ⇒ Horde Base
Summary ⇒ Currencies and Taxes
Type ⇒ Enhancement
Reply to this comment
Mini module for manage Currencies and Taxes.

Meant to access their data from other Apps in order to avid 
duplication of code and data.

Handles locale price formating too.


Saved Queries