El dom, 01-08-2004 a las 07:25, Jani tiainen escribió: > lincoln phipps openmutual net wrote: > > > > > > Alvaro del Castillo wrote: > > > >> Hi guys! > >> > >> After reading the Lincoln and Jani emails, and coding and testing 2 > >> hours, here goes a new patch with the database version for all planner > >> projects in a separate table. With this approach, we can grow the table > >> to meet Liconln hosting needs and also, Jani ideas. > >> > > > > OK - I see how you've done that and I'm happy. Once things settle > > down then I can easily hack your code and add new features to it. > > > > Also there are a minor spelling mistake in "propecty", > > > > > >> RCS file: /cvs/gnome/planner/libplanner/mrp-sql.c,v > > > > > >> + g_warning ("DECLARE CURSOR command failed (propecty_type) %s.", > > > > > > > > Also with, > > gchar *init_database = "template1"; > > > > why change this name and why not keep what the user provided > > when they tried to access the database ? ...or if this is > > the first use then why not use the same name as provided by the > > user (where they get asked for host, databasename, username, password). > > I don't understand why the init_database is hard coded as template1. > > > > The dialog...., > > > >> + dialog = gtk_message_dialog_new (window, > >> + GTK_DIALOG_DESTROY_WITH_PARENT, > >> + GTK_MESSAGE_QUESTION, > >> + GTK_BUTTONS_NONE, > >> + _("Database %s need to be upgraded " > >> + "from version %s to version %s."), > >> + database_name, database_version, > >> + max_version_upgrade); > > > > > > should have "needs" not "need" as there is just one database (weird > > English language !). > > > > Actually there should be advice to backup database before upgrade - you > never know if something goes wrong. Yes, in a previous version I think we have it. I have changed it to: _("Database %s needs to be upgraded " "from version %s to version %s.\n" "Please backup database before the upgrade."), Cheers -- Alvaro
Attachment:
signature.asc
Description: Esta parte del mensaje =?ISO-8859-1?Q?est=E1?= firmada digitalmente