[OSRAIDS-developers] Re: Tempory set-up

[<osr...@li...>]

OK I am looking at all the code you sent me.

have most of it in.

some issues 
########################################################
Date
########
 - I think that changing it at the report level is not the best of ideas
so here are three ideas about a fix. My choice is the third and the function
for calculating invoice due dates will need revision. We could also write a 
function for this or I could email you my work on the calendar module. Yes
I have hidden code from you and the users. I want to work more on the 2.0 branch
but the I18N and css inspired me to work some on the 1.x.x branch.

1. in the sql syntax - osraids is using the sql engine already to format 
some things so it is better to utilize the same process for the conversion
(hint: I won an award for using the database to format data 
when the scripting language was slow, ASP in 1998)

2. the date formating should work in all forms and there for have a function
to do this so that all forms can display the way you wish.

3. from setup on you could declare the structure by changing the database
table creation commands once.  -  my vote for easiest !

< //	$pdf->Cell(2,0.25,$dbrow["SerDate"],0,2,'L');
< 	$serdate = strtotime( $dbrow["SerDate"]);
< 	$serdate = date ($datefmt,$serdate );
< 	$pdf->Cell(2,0.25,$serdate,0,2,'L');	
---
> 	$pdf->Cell(2,0.25,$dbrow["SerDate"],0,2,'L');
#########
Variables
#########
This can get dangerous. I will set it right and email you. The long
trem issue is that the lang array will be replaced by a database
driven array that will not contain any other languages. so I will fix
a select box for the config.
 
< $confinfo .= "\$clang = \"" . $lang . "\";\n";
< $confinfo .= "\$lang = $$lang;\n";
 
########################################################

more as I go.... spent to much time with the TV tonight.... Election and all....


On Tue, 2 Nov 2004 22:47:57 +0000, Martin Owen <ma...@re...> wrote:
> Andrew, as a temp setup till the wizard is done, I've changed the
> config/form.php and config/createconf.php diff's attached.
> 
> MRO...
> 
> 
> 


-- 
Andrew Latham

http://www.lathama.com
la...@gm...
la...@ya...
la...@la...
If any of the above are not working we have 
bigger problems than my email.