[PATCH] Re: Fancy Invoice
On Mon, 2006-06-02 at 18:59 -0800, Brian wrote: > > There is one more change I would like to make. Add a parameter to set > the minimum number of entries in the body rather than it be hard coded > in the scheme file. > I've finished making the changes. I don't think I've missed any of the important hard coded items in the fancy-invoice. You may want to clean up a few of the old statements I've commented out in favour of my changes. You might also want to check that I have all the translation markings correct. I followed the code that was there, but some of that may be wrong since he states that it might be in his oli-custom comments. I also think these changes could be applied to the 1.8.12 as well since I started them there quite a while ago. Side note: the gnc:make-number-range-option also does not render on the screen the best. It seems to have a display box slightly shorter than the number text it is displaying. I had to compensate by making the minimum display number width wider. -- Brian <[EMAIL PROTECTED]> Index: src/business/business-reports/fancy-invoice.scm === --- src/business/business-reports/fancy-invoice.scm (revision 13138) +++ src/business/business-reports/fancy-invoice.scm (working copy) @@ -43,6 +43,9 @@ ;; template. The most common used templates will be distributed with ;; gnucash. +;; Modifed to use settable options instead of the hard coded ones. +;; modified by Brian Dolbec Feb. 6, 2006 + (define-module (gnucash report fancy-invoice)) (use-modules (srfi srfi-1)) @@ -242,7 +245,7 @@ (define (options-generator) (define gnc:*report-options* (gnc:new-options)) - + (define (gnc:register-inv-option new-option) (gnc:register-option gnc:*report-options* new-option)) @@ -331,14 +334,31 @@ "tc" (N_ "Display the payments applied to this invoice?") #f)) (gnc:register-inv-option + (gnc:make-number-range-option +(N_ "Display") (N_ "Minimum # of entries") +"u" (N_ "The minimum number of invoice entries to display. (-1)") 23 +4 23 3 1)) + + (gnc:register-inv-option (gnc:make-text-option (N_ "Display") (N_ "Extra Notes") "u" (N_ "Extra notes to put on the invoice") ;; oli-custom - Extra notes to add on each invoice, invoice-independent - ;; yes, I was too lazy to (get-company-name) ;) - "Make all cheques payable to: Company Name Inc.\nDirect all inquiries to: Mr. Accounting Contact")) + ;; yes, I was too lazy to (get-company-name) ;) + ;;"Make all cheques payable to: Company Name Inc.\nDirect all inquiries to: Mr. Accounting Contact")) + "")) (gnc:register-inv-option + (gnc:make-simple-boolean-option +(N_ "Display") (N_ "Payable to:") + "ua" (N_ "Display the Payable to: information") #t)) + + (gnc:register-inv-option + (gnc:make-simple-boolean-option +(N_ "Display") (N_ "Company contact") + "ub" (N_ "Display the Company contact information") #t)) + + (gnc:register-inv-option (gnc:make-string-option (N_ "Display") (N_ "Today Date Format") "v" (N_ "The format for the date->string conversion for today's date.") @@ -440,7 +460,7 @@ (do ((entries-added entries-added (+ entries-added 1)) (odd-row? odd-row? (not odd-row?))) ;; oli-custom - here you put the minimum number of rows minus one - ((> entries-added 23)) + ((> entries-added (opt-val "Display" "Minimum # of entries" ))) (gnc:html-table-append-row/markup! table (if odd-row? "normal-row" "alternate-row") (string->list (make-string (num-columns-required used-columns) #\space))) ) (add-subtotal-row table used-columns value-collector @@ -616,15 +636,34 @@ 'attribute (list "valign" "top")) table)) +;; define a couple variables used in various places +;;(define c-name gnc:*company-name* ) +;;(define c-contact gnc:*company-contact*) + (define (make-myname-table book date-format) (let* ((table (gnc:make-html-table)) (slots (gnc:book-get-slots book)) (name (gnc:kvp-frame-get-slot-path slots (append gnc:*kvp-option-path* (list gnc:*business-label* gnc:*company-name* +;; (contact (gnc:kvp-frame-get-slot-path +;; slots (append gnc:*kvp-option-path* +;; (list gnc:*business-label* gnc:*company-contact* (addy (gnc:kvp-frame-get-slot-path slots (append gnc:*kvp-option-path* (list gnc:*business-label* gnc:*company-addy* + (id (gnc:kvp-frame-get-slot-path + slots (append gnc:*kvp-option-path* + (list gnc:*business-label* gnc:*company-id* + (phone (gnc:kvp-frame-get-slot-path + slots (append gnc:*kvp-option-path* + (list gnc:*business-label* gnc:*company-phone* + (fax (gnc:kvp-frame-get-slot-path + slots (append gnc:*kvp-option-path* + (list gnc:*business-label* gnc:*company-fax* + (url (gnc:kvp-frame-get-slot-path + slots (append gnc:*kvp-option-path* + (list gnc:*business-label* gnc:*company-url*
Re: problems with g2
On Mon, 2006-02-06 at 19:34 -0700, Mark Johnson wrote: > This time I got Rev 13130. > > By my count, I see four errors reported in my first e-mail. > > First, I am still getting the dialog about gnucash not being set up > ("Cannot find default values"). This dialog appears EVERY time I start > gnucash. I have now submitted this as bug #330203. I was a little > surprised not to find this in bugzilla as I thought I had seen it > mentioned on the list a time or two. This should have been fixed by r13104. Please remove your ~/.gconf.path file and try again. David ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: Fancy Invoice
On Mon, 2006-06-02 at 16:34 -0500, Derek Atkins wrote: > What you > CAN do is add two checkboxes to let the user decide whether to display the > "Direct all inquiries to: c-cname" and "Make all cheques payable to: > c-contact" > messages. Then you leave the two strings in the options (so the user > can re-define the message) but then you build the actual report based > on the checkbox, the contents of the option string, and the c-{name,contact} > during the invoice run. > > Unfortunately there's just no way to seed c-contact and c-name in the options > creation because they exist outside the book setting. > > > I'll finish cleaning it up and generate a patch. :) > > -derek Done, Thank you Derek. I knew one of you would know how to do it. It just took me all afternoon to actually accomplish it :) (countless #!$% #@ trial and errors later) Attached is the diff implementing all the changes. There is one more change I would like to make. Add a parameter to set the minimum number of entries in the body rather than it be hard coded in the scheme file. Another wish is to be able to set the default invoice type for printing. i.e. choose between {printable, easy, fancy-invoice, etc.} as part of the business options. -- Brian <[EMAIL PROTECTED]> Index: src/business/business-reports/fancy-invoice.scm === --- src/business/business-reports/fancy-invoice.scm (revision 12055) +++ src/business/business-reports/fancy-invoice.scm (working copy) @@ -242,7 +242,7 @@ (define (options-generator) (define gnc:*report-options* (gnc:new-options)) - + (define (gnc:register-inv-option new-option) (gnc:register-option gnc:*report-options* new-option)) @@ -335,10 +335,21 @@ (N_ "Display") (N_ "Extra Notes") "u" (N_ "Extra notes to put on the invoice") ;; oli-custom - Extra notes to add on each invoice, invoice-independent - ;; yes, I was too lazy to (get-company-name) ;) - "Make all cheques payable to: Company Name Inc.\nDirect all inquiries to: Mr. Accounting Contact")) + ;; yes, I was too lazy to (get-company-name) ;) + ;;"Make all cheques payable to: Company Name Inc.\nDirect all inquiries to: Mr. Accounting Contact")) + "")) (gnc:register-inv-option + (gnc:make-simple-boolean-option +(N_ "Display") (N_ "Payable to:") + "ua" (N_ "Display the Payable to: information") #t)) + + (gnc:register-inv-option + (gnc:make-simple-boolean-option +(N_ "Display") (N_ "Company contact") + "ub" (N_ "Display the Company contact information") #t)) + + (gnc:register-inv-option (gnc:make-string-option (N_ "Display") (N_ "Today Date Format") "v" (N_ "The format for the date->string conversion for today's date.") @@ -616,15 +627,34 @@ 'attribute (list "valign" "top")) table)) +;; define a couple variables used in various places +;;(define c-name gnc:*company-name* ) +;;(define c-contact gnc:*company-contact*) + (define (make-myname-table book date-format) (let* ((table (gnc:make-html-table)) (slots (gnc:book-get-slots book)) (name (gnc:kvp-frame-get-slot-path slots (append gnc:*kvp-option-path* (list gnc:*business-label* gnc:*company-name* +;; (contact (gnc:kvp-frame-get-slot-path +;; slots (append gnc:*kvp-option-path* +;; (list gnc:*business-label* gnc:*company-contact* (addy (gnc:kvp-frame-get-slot-path slots (append gnc:*kvp-option-path* (list gnc:*business-label* gnc:*company-addy* + (id (gnc:kvp-frame-get-slot-path + slots (append gnc:*kvp-option-path* + (list gnc:*business-label* gnc:*company-id* + (phone (gnc:kvp-frame-get-slot-path + slots (append gnc:*kvp-option-path* + (list gnc:*business-label* gnc:*company-phone* + (fax (gnc:kvp-frame-get-slot-path + slots (append gnc:*kvp-option-path* + (list gnc:*business-label* gnc:*company-fax* + (url (gnc:kvp-frame-get-slot-path + slots (append gnc:*kvp-option-path* + (list gnc:*business-label* gnc:*company-url* (invoice-cell (gnc:make-html-table-cell)) (name-cell (gnc:make-html-table-cell)) @@ -658,8 +688,11 @@ 'attribute (list "align" "right") 'attribute (list "width" "33%")) (gnc:html-table-append-row! table (list (string-expand - (if addy addy "") - #\newline "") "Phone: (111) 222-Web: http://companysite.com"; "")) + (if addy addy "") #\newline "") + (string-expand (string-append (_ "Phone: ") (if phone phone "") + (if fax (string-append "\n" (_ "Fax: ") fax) "")) #\newline "" ) + (if url (string-append (_ "Web: ") url) ""))) + ;; oli-custom - I didn't want today's date on the invoice. The invoice already has a date. ;; Today's date can be in the email, fax or letter accompanying the invoice. ;;(gnc:html-table-append-row! table (list @@ -711,7 +744,8 @@ ;;(gnc:html-document-set-title! document title) (if i
Re: problems with g2
This time I got Rev 13130. By my count, I see four errors reported in my first e-mail. First, I am still getting the dialog about gnucash not being set up ("Cannot find default values"). This dialog appears EVERY time I start gnucash. I have now submitted this as bug #330203. I was a little surprised not to find this in bugzilla as I thought I had seen it mentioned on the list a time or two. Second, the numbers do not disappear on me. This problem is history. Third, I am no longer getting errors about being unable to parse the URL on File->Save As. Fourth, this (quick-fill problem) has been entered as bug #328893. From the comments, this is making progress. It is caused by what version of gnucash created the file and not by the version of the libraries being used. Mark Derek Atkins wrote: Can you update to 13129 and try again? If you can still do this then please file a bug report in bugzilla. I know that there were fixes for the gconf settings. I don't know about the File Save-As. -derek Mark Johnson <[EMAIL PROTECTED]> writes: Hi, I built and installed g2 by: make distclean svn co http:/svn.gnucash.org/repo/gnucash/trunk ./autogen.sh ./configure --enable-opt-style-install --prefix=/opt/gnucash-svn make su make install The revision obtained was 12327. I changed to the directory /opt/gnucash-svn/bin and ran gnucash from there (the directory is not on my PATH). When I started it, it put up a druid for some configuration. I did not note the specifics of this, but it was something to do with a path in .gconf. I was able to create a few accounts, but when I tried to enter transactions, the numbers would disappear when I left the cells by either tabbing or pressing enter. When I selected "File-Save As", it allowed me to create a new directory in my home, I entered a file name and clicked OK. I got an error stating that it was unable to parse the URL; the URL being the path and file name to which I had selected saving the data. It had nothing like "file://" on the beginning. It was simply "/mnt/raid/home/mj/gnucash-svn12327/books.xac". Both these issues seem too obvious not to have been noticed. Any ideas? In 1.8.12, when selecting an account for a split in the register, pressing ":" would complete the account name and allow one to select an account lower in the hierarchy. This no longer worked. One really notices the lack of convenience when it is gone. Mark ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: [PATCH] Improve right justified cells in register
On Mon, 2006-02-06 at 20:13 -0500, Scott Oonk wrote: > The attached patch fixes a couple of problems with right justified > cells in the register. Applied. David ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
[PATCH] Improve right justified cells in register
Hi, The attached patch fixes a couple of problems with right justified cells in the register. - Make sure the text in the cell stays right justified when deleting characters. - Adjust the positioning of text to account for the popup toggle. - Change gnucash-item-edit to use logical extents instead of ink extents. This syncs gnucash-item-edit.c with change made to gnucash-grid.c in r12155. Scott Index: src/register/register-gnome/gnucash-item-edit.c === --- src/register/register-gnome/gnucash-item-edit.c (revision 13090) +++ src/register/register-gnome/gnucash-item-edit.c (working copy) @@ -118,7 +118,52 @@ *y += yd; } +static void +gnc_item_edit_update_offset (GncItemEdit *item_edit, TextDrawInfo *info) +{ +gint drawable_width; +gint visible_width; +PangoRectangle logical_rect; +g_return_if_fail (item_edit != NULL); +g_return_if_fail (GNC_IS_ITEM_EDIT (item_edit)); + +pango_layout_get_pixel_extents (info->layout, NULL, &logical_rect); + +drawable_width = info->text_rect.width - 2 * CELL_HPADDING; + +// Layout is smaller than drawable area, or we've entered a +// new cell. Use default x_offset. +if (logical_rect.width <= drawable_width || item_edit->reset_pos) +{ +gnc_item_edit_reset_offset (item_edit); +} + +// Layout is wider than drawable area +if (logical_rect.width > drawable_width) +{ +//Make sure cursor is inside the drawn area +if (info->cursor_rect.x + item_edit->x_offset > +info->text_rect.x + drawable_width) +{ +item_edit->x_offset = (info->text_rect.x + drawable_width) +- info->cursor_rect.x; +} +else if (info->cursor_rect.x + item_edit->x_offset < info->text_rect.x) +{ +item_edit->x_offset = - info->cursor_rect.x; +} + +// Make sure the entire drawable area is filled. +visible_width = logical_rect.width + item_edit->x_offset; + +if (visible_width < drawable_width) +{ +item_edit->x_offset += (drawable_width - visible_width); +} +} +} + static void gnc_item_edit_draw_info (GncItemEdit *item_edit, int x, int y, TextDrawInfo *info) { @@ -218,13 +263,6 @@ // pango_layout_set_ellipsize(...) as of pango 1.6 may be useful for // strings longer than the field width. -// Default x_offset based on cell alignment -if (item_edit->reset_pos) -{ -gnc_item_edit_reset_offset (item_edit); -item_edit->reset_pos = FALSE; -} - pango_layout_get_cursor_pos (info->layout, cursor_byte_pos, &strong_pos, NULL); info->cursor_rect.x = dx + PANGO_PIXELS (strong_pos.x); @@ -239,6 +277,8 @@ info->hatch_rect.width = wd; info->hatch_rect.height = hd; } + +gnc_item_edit_update_offset (item_edit, info); } static void @@ -251,22 +291,6 @@ } static void -gnc_item_edit_update_scroll_offset(GncItemEdit *item_edit, - TextDrawInfo *info) { -if (info->cursor_rect.x + item_edit->x_offset > -info->text_rect.x + info->text_rect.width - CELL_HPADDING) -{ -item_edit->x_offset = -(info->text_rect.x + info->text_rect.width - CELL_HPADDING) -- info->cursor_rect.x; -} -else if (info->cursor_rect.x + item_edit->x_offset < info->text_rect.x) -{ -item_edit->x_offset = - info->cursor_rect.x; -} -} - -static void gnc_item_edit_draw (GnomeCanvasItem *item, GdkDrawable *drawable, int x, int y, int width, int height) { @@ -299,8 +323,6 @@ gdk_gc_set_foreground (item_edit->gc, info.fg_color); - gnc_item_edit_update_scroll_offset(item_edit, &info); - gdk_draw_layout (drawable, item_edit->gc, info.text_rect.x + CELL_HPADDING + @@ -569,18 +591,26 @@ gnc_item_edit_reset_offset (GncItemEdit *item_edit) { Table *table; -PangoRectangle ink_rect; +PangoRectangle logical_rect; PangoLayout *layout; gint x, y, width, height; +gint drawable_width; +gint toggle_space; g_return_if_fail (item_edit != NULL); g_return_if_fail (GNC_IS_ITEM_EDIT(item_edit)); table = item_edit->sheet->table; layout = gtk_entry_get_layout (GTK_ENTRY(item_edit->editor)); -pango_layout_get_pixel_extents (layout, &ink_rect, NULL); + +pango_layout_get_pixel_extents (layout, NULL, &logical_rect); gnc_item_edit_get_pixel_coords (item_edit, &x, &y, &width, &height); +tog
Tests and RNG
I have seen mentioned a number of times were the inital seed for the RNG can effect the outcome of the test because under some conditions it may trigger a bug and that the current test infrastructure does not help in being able to reproduce these problems. I would like to suggest that test be change to take an optional seed and to report the seed being used. The default behavior is not changed allowing a wide range of random conditions to be tested but when a test does fail the seed used can be found in the logs and provided to allow reporoducing the problem. Regards, Chris Leach ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: [Gnucash-changes] r13135 - gnucash/trunk/src/backend/postgres
On Monday 06 February 2006 9:33 pm, Chris Shoemaker wrote: > > Log: > > more inst.kvp_data changes to get Postgres backend to compile > > I apologize for the collateral damage here, Neil. Thanks for fixing > it. No problem. It's nothing I haven't done myself before now. > I've now sync'd my configure script between my devel machines so > that I'm bulding the postgres backend everywhere. And for much the same reason. :-) -- Neil Williams = http://www.data-freedom.org/ http://www.nosoftwarepatents.com/ http://www.linux.codehelp.co.uk/ pgpiJlriQrTp1.pgp Description: PGP signature ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: Fancy Invoice
Quoting Brian <[EMAIL PROTECTED]>: in the invoice options there is: (Partially modified code) (gnc:register-inv-option (gnc:make-text-option (N_ "Display") (N_ "Extra Notes") "u" (N_ "Extra notes to put on the invoice") ;; oli-custom - Extra notes to add on each invoice, invoice-independent ;; yes, I was too lazy to (get-company-name) ;) (string-append (_ "Make all cheques payable to: ") ;;c-name (_ "\nDirect all inquiries to: " ;;c-contact))) ;;"Make all cheques payable to: Company Name Inc.\nDirect all inquiries to: Mr. Accounting Contact")) So far I've not been able to get the c-name & c-contact from the book. This module seems to run before the make-myname-table procedure. The book parameter is not accessible at this point. I have been far too busy this past 2 months to work on it and figure it out. Perhaps you or someone that is good at scheming could fix it easily. Ahh... Okay, a little background. The report options are really just a template. They are "outside" the book, and exist outside of it. The templates exist (and are filled in) without access to a book, so you cannot pull options from the book interface to put them into the report options. In other words, there really isn't a way to do what you want.. What you CAN do is add two checkboxes to let the user decide whether to display the "Direct all inquiries to: c-cname" and "Make all cheques payable to: c-contact" messages. Then you leave the two strings in the options (so the user can re-define the message) but then you build the actual report based on the checkbox, the contents of the option string, and the c-{name,contact} during the invoice run. Unfortunately there's just no way to seed c-contact and c-name in the options creation because they exist outside the book setting. I'll finish cleaning it up and generate a patch. :) -derek -- Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory Member, MIT Student Information Processing Board (SIPB) URL: http://web.mit.edu/warlord/PP-ASEL-IA N1NWH [EMAIL PROTECTED]PGP key available ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: [Gnucash-changes] r13135 - gnucash/trunk/src/backend/postgres - more inst.kvp_data changes to get Postgres backend to compile
On Mon, Feb 06, 2006 at 03:45:28PM -0500, Neil Williams wrote: > Author: codehelp > Date: 2006-02-06 15:45:27 -0500 (Mon, 06 Feb 2006) > New Revision: 13135 > Trac: http://svn.gnucash.org/trac/changeset/13135 > > Modified: >gnucash/trunk/src/backend/postgres/PostgresBackend.c >gnucash/trunk/src/backend/postgres/txn.c >gnucash/trunk/src/backend/postgres/txnmass.c > Log: > more inst.kvp_data changes to get Postgres backend to compile I apologize for the collateral damage here, Neil. Thanks for fixing it. I've now sync'd my configure script between my devel machines so that I'm bulding the postgres backend everywhere. -chris ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: Fancy Invoice
On Mon, 2006-06-02 at 15:42 -0500, Derek Atkins wrote: > Quoting Brian <[EMAIL PROTECTED]>: > > > I have a number of changes completed for the fancy invoice. I have > > added properties to the File=>Properties dialog and that info is saved. > > I have added Phone,Fax,Website properties to the invoice. > > Excellent. > > > What I have not been able to do yet is get the info into the extra notes > > from the book. My scheme abilities are not that good :(. > > I'm not sure what you mean by "get the info into the extra notes from > the book". What extra notes? > > > I'll make a patch with what I have so far probably later today. I need > > to clean out some dead code I was experimenting with (which doesn't > > work). > > Okay. I /suspect/ what you did was just add "options" in the report > to hold these fields instead of just grabbing them from the book. At > least that's what I think you did based upon my interpretation of what > you meant above by the "extra notes" issue. > I've added them to the book as I stated above. I have also retrieved them from the book and added them to the report. > If you look at the make-myname-table procedure in invoice.scm, that's > where it pulls the company name and address from the book properties. > > -derek > That is where most of them are, those weren't that hard (with a little help from you), but not all. in the invoice options there is: (Partially modified code) (gnc:register-inv-option (gnc:make-text-option (N_ "Display") (N_ "Extra Notes") "u" (N_ "Extra notes to put on the invoice") ;; oli-custom - Extra notes to add on each invoice, invoice-independent ;; yes, I was too lazy to (get-company-name) ;) (string-append (_ "Make all cheques payable to: ") ;;c-name (_ "\nDirect all inquiries to: " ;;c-contact))) ;;"Make all cheques payable to: Company Name Inc.\nDirect all inquiries to: Mr. Accounting Contact")) So far I've not been able to get the c-name & c-contact from the book. This module seems to run before the make-myname-table procedure. The book parameter is not accessible at this point. I have been far too busy this past 2 months to work on it and figure it out. Perhaps you or someone that is good at scheming could fix it easily. I'll finish cleaning it up and generate a patch. :) -- Brian <[EMAIL PROTECTED]> ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
GConf and $(HOME) on OSX
A common source of problems on OSX could be incompletely configured environments - I've documented those that need to be set to build gnucash but the $(HOME) variable could catch out users as if it is not set, OSX loads the GConf druid every single time gnucash starts. Whereas g_get_home_dir is used later, this define could be a problem if the environment is incomplete: #define PATH_STRING1 "xml:readwrite:$(HOME)/.gconf\n" src/gnome-utils/druid-gconf-setup.c I can't see it's our fault if HOME isn't set but it might be wise to know in advance. -- Neil Williams = http://www.data-freedom.org/ http://www.nosoftwarepatents.com/ http://www.linux.codehelp.co.uk/ pgpV7afVzqcqq.pgp Description: PGP signature ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: [Gnucash-changes] r13132 - gnucash/trunk/src/engine - Keep QOF aware of the dirty-state of Transactions.
Quoting Chris Shoemaker <[EMAIL PROTECTED]>: On Mon, Feb 06, 2006 at 03:37:11PM -0500, Derek Atkins wrote: So it's somewhat questionable at this point if we want splits to be dirtiable or not. At some level, yea, they should be. Yeah, I'd say this is true at the persistence-level. I.e. I really don't have to have to read/write the whole trans + all splits when only the description of one split has changed. I'll ponder this for a bit... But at another, we necessarily WANT splits to be committed or rolled back together with a full transaction, not individually.. So we DO want it based on xaccTransBeginEdit/CommitEdit, and not introduce an xaccSplitBeginEdit/CommitEdit. Yeah, I'd call this the engine-API level. Committing _just_ a split doesn't and shouldn't make sense at the engine-API level. Okay, that's certainly fair.. Just so long as it's well documented where it belongs, who calls what, etc. We need a well defined calling invariant as well as a clear event model upwards to the UI (GncEvent) and down to the backend (begin/commit-edit). I suspect that's also why the Split used the QofEntity instead of QofInstance, because the Split didn't want its own last_update or dirty flag -- it's inherited from the Transaction. I think it's better to allow Splits as first-class objects. Then we at least have the _option_ of using a DB schema that treats Splits separately from Transactions. ISTM, that's the most reasonable way of handling Transactions with variable numbers of Splits. How would the existing SQL handle more than 2 Splits per trans? The API only handled transactions; the SQL code stored splits in their own table. So at the DB Schema level, yes, splits were still first class objects. But at the Engine API level they are (were?) not. -chris -derek -- Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory Member, MIT Student Information Processing Board (SIPB) URL: http://web.mit.edu/warlord/PP-ASEL-IA N1NWH [EMAIL PROTECTED]PGP key available ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: No Subject
Am Montag, den 06.02.2006, 07:51 -0800 schrieb Joshua Facemyer: > I'm trying to build svn gnucash on gentoo. I get this error repeatedly: > > configure:26767: error: Library requirements (libgsf-1 >= 1.12.2 > libgsf-gnome-1 >=1.12.2) not met; consider adjusting the PKG_CONFIG_PATH > environment variable if your libraries are in a nonstandard prefix so > pkg-config can find them. > > I've tried to re-emerge libgsf with "gnome" use flag enabled, but it > doesn't build libgsf-gnome at all. But it should install libgsf-gnome. I have installed libgsf-1.13.3 here, and it works fine. Maybe you can try it again? Martin > I don't run gnome, so otherwise all my apps are compiled -gnome, which is > why I enable the use flag for libgsf. > > I know this is probably a libgsf problem, but thought maybe someone else > has had this same problem - any clue? > > Joshua Facemyer > > > - > Catholic Financial Services > Send and Receive all online Payments And Donations > No Merchant Account Required! > https://www.catholicfs.org/ > ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: [Gnucash-changes] r13132 - gnucash/trunk/src/engine - Keep QOF aware of the dirty-state of Transactions.
On Mon, Feb 06, 2006 at 03:37:11PM -0500, Derek Atkins wrote: > So it's somewhat questionable at this point if we want splits to be > dirtiable > or not. At some level, yea, they should be. Yeah, I'd say this is true at the persistence-level. I.e. I really don't have to have to read/write the whole trans + all splits when only the description of one split has changed. > But at another, we necessarily > WANT splits to be committed or rolled back together with a full transaction, > not individually.. So we DO want it based on xaccTransBeginEdit/CommitEdit, > and not introduce an xaccSplitBeginEdit/CommitEdit. Yeah, I'd call this the engine-API level. Committing _just_ a split doesn't and shouldn't make sense at the engine-API level. > I suspect that's also why the Split used the QofEntity instead of > QofInstance, > because the Split didn't want its own last_update or dirty flag -- it's > inherited from the Transaction. I think it's better to allow Splits as first-class objects. Then we at least have the _option_ of using a DB schema that treats Splits separately from Transactions. ISTM, that's the most reasonable way of handling Transactions with variable numbers of Splits. How would the existing SQL handle more than 2 Splits per trans? -chris ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: Fancy Invoice
Quoting Brian <[EMAIL PROTECTED]>: I have a number of changes completed for the fancy invoice. I have added properties to the File=>Properties dialog and that info is saved. I have added Phone,Fax,Website properties to the invoice. Excellent. What I have not been able to do yet is get the info into the extra notes from the book. My scheme abilities are not that good :(. I'm not sure what you mean by "get the info into the extra notes from the book". What extra notes? I'll make a patch with what I have so far probably later today. I need to clean out some dead code I was experimenting with (which doesn't work). Okay. I /suspect/ what you did was just add "options" in the report to hold these fields instead of just grabbing them from the book. At least that's what I think you did based upon my interpretation of what you meant above by the "extra notes" issue. If you look at the make-myname-table procedure in invoice.scm, that's where it pulls the company name and address from the book properties. -derek -- Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory Member, MIT Student Information Processing Board (SIPB) URL: http://web.mit.edu/warlord/PP-ASEL-IA N1NWH [EMAIL PROTECTED]PGP key available ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: [Gnucash-changes] r13132 - gnucash/trunk/src/engine - Keep QOF aware of the dirty-state of Transactions.
Quoting Chris Shoemaker <[EMAIL PROTECTED]>: I admit I haven't looked at the code recently.. Part of the issue is that in XML the Splits aren't listed on their own -- they only exist within a Transaction -- so you kinda need to mark the transaction as dirty when you change a split. Yeah, I don't know why, but, looking at the code more closely, we don't even actually keep dirty state on Splits at _all_. mark_split() just flags the account _balance_ and _sorting_ as dirty. I don't see any good reason not to actually keep dirty state for the splits. Then you can ask a transaction if it has dirty splits, something we can't currently do. It's somewhat historical. Obviously none of this really matters for the xml file backend. In the current SQL backend, the interface really is (was?) limited to Accounts and Transactions. There were no "Splits" in the interface. You stored a transaction en-masse, not a split. So a "dirty split" didn't matter; a split was just a part of a transaction and couldn't live by itself. That's why we never dirtied splits; a dirty split just dirties the transaction and that way the full transaction is written to the DB and accounts can be updated etc. So it's somewhat questionable at this point if we want splits to be dirtiable or not. At some level, yea, they should be. But at another, we necessarily WANT splits to be committed or rolled back together with a full transaction, not individually.. So we DO want it based on xaccTransBeginEdit/CommitEdit, and not introduce an xaccSplitBeginEdit/CommitEdit. I suspect that's also why the Split used the QofEntity instead of QofInstance, because the Split didn't want its own last_update or dirty flag -- it's inherited from the Transaction. I'm not sure in what cases you ever want to or need to mark a split dirty just because the transaction changed. Well, like I said, the two existing cases are actually just dirtying computed-state (not stored-state) in the Account. I think they're both correct: changing the trans date, and the trans currency. But, I don't think these cases should ever dirty the stored-state of the Account, and I think it should dirty the stored-state of the Split iff the split value has to be re-expressed in a new denominator. Changing the trans date or trans currency need not dirty the account stored state. However they obviously need to dirty the txn stored state. (I'll note that we still need the xaccAccountInsertSplit(), but I think that only should be called "interally" by xaccSplitSetAccount(), and it doesn't necessarily need a begin/commit edit because it's only structural data, not real data). Exactly. I'd be more emphatic. s/doesn't necessarily/really shouldn't/ :) Sure.. :) -chris -derek -- Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory Member, MIT Student Information Processing Board (SIPB) URL: http://web.mit.edu/warlord/PP-ASEL-IA N1NWH [EMAIL PROTECTED]PGP key available ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: Fancy Invoice
On Mon, 2006-06-02 at 12:48 -0500, Derek Atkins wrote: > Quoting Andrew Sackville-West <[EMAIL PROTECTED]>: > > > So what is the proper way to look up that specific info from the book > > database? > > Well, I think the phone number and website would need to be added to > the book properties (take a look in business-prefs.scm). Then you > can load the book properties and lookup the contents (see invoice.scm > for an example of how it pulls in the company name and address). > > > A > > -derek > I have a number of changes completed for the fancy invoice. I have added properties to the File=>Properties dialog and that info is saved. I have added Phone,Fax,Website properties to the invoice. What I have not been able to do yet is get the info into the extra notes from the book. My scheme abilities are not that good :(. I'll make a patch with what I have so far probably later today. I need to clean out some dead code I was experimenting with (which doesn't work). -- Brian <[EMAIL PROTECTED]> ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: [Gnucash-changes] r13132 - gnucash/trunk/src/engine - Keep QOF aware of the dirty-state of Transactions.
On Mon, Feb 06, 2006 at 01:47:12PM -0500, Derek Atkins wrote: > Quoting Chris Shoemaker <[EMAIL PROTECTED]>: > > >mark_trans() is horribly named. It's a fundamentally different > >operation than qof_instance_set_dirty(). _Most_ of the places we call > >qof_instance_dirty (11 of them) we don't really want to dirty the > >splits. > > I admit I haven't looked at the code recently.. Part of the issue is that > in XML the Splits aren't listed on their own -- they only exist within > a Transaction -- so you kinda need to mark the transaction as dirty when > you change a split. Yeah, I don't know why, but, looking at the code more closely, we don't even actually keep dirty state on Splits at _all_. mark_split() just flags the account _balance_ and _sorting_ as dirty. I don't see any good reason not to actually keep dirty state for the splits. Then you can ask a transaction if it has dirty splits, something we can't currently do. > I'm not sure in what cases you ever want to or need to mark a split > dirty just because the transaction changed. Well, like I said, the two existing cases are actually just dirtying computed-state (not stored-state) in the Account. I think they're both correct: changing the trans date, and the trans currency. But, I don't think these cases should ever dirty the stored-state of the Account, and I think it should dirty the stored-state of the Split iff the split value has to be re-expressed in a new denominator. > >>On a separate (but related) note, I want to separate extend the events > >>in order to differentiate between "Account data modified" and "account > >>contents modified" -- where the former is emitted by the new/edit account > >>dialog, and the latter is emitted by the register... > > > >Absolutely, and this is _very_ related, because the same case applies > >to Transactions. The mark_trans() cases should correlate with > >emitting the "child_changed" signal (or whatever), while the > >qof_instance_set_dirty() cases should correlate with emitting just the > >"changed" signal (or whatever). And in two places, we should emit both. > > We need to come up with a good set of signals here. "MODIFIED" isn't > sufficiently robust. :) Unfortunately "INSERTED" and "REMOVED" are > already used, and used IMHO in weird ways. :( Definitely. > >This is a very useful distinction for both Accounts and Transactions. > > ... and Invoices. > > Indeed, another question is who owns the link. For example, I think > the Split<->Account mapping is backwards.. Right now you need to > Begin/Commit > on the Account, but it's really the Split that contains the mapping. > The XML (and SQL) data for an account does not keep track of the splits. > The split keep track of the account. So, shouldn't it be > xaccSplitSetAccount() > instead of xaccAccountInsertSplit(), and shouldn't that be wrapped around > a Split/Trans Begin/Commit edit? Yeah, I think you're right. I can't think of any case where modifying a Split should require a begin/commit edit on an Account. > (I'll note that we still need the xaccAccountInsertSplit(), but I think that > only should be called "interally" by xaccSplitSetAccount(), and it doesn't > necessarily need a begin/commit edit because it's only structural data, > not real data). Exactly. I'd be more emphatic. s/doesn't necessarily/really shouldn't/ :) -chris ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: At revision 13129: error reading gnucash file
On Monday 06 February 2006 20:23, David Hampton wrote: > On Mon, 2006-02-06 at 19:36 +0100, Eildert Groeneveld wrote: > > On Monday 06 February 2006 17:53, Derek Atkins wrote: > > > Anything else printed on the terminal? > > > What about in /tmp/gnucash.trace? > > > > alright, this is what I see there: > > > > > > loading stylesheets > > looking for /home2/eg/.gnucash/stylesheets-2.0 > > trying to load /home2/eg/.gnucash/stylesheets-2.0 > > Invalid timestamp in data file. Look for a 'price:time' entry with a > > date of 1969-12-31 or 1970-01-01. > > > > for loglevel 2 > > > > does this help? > > That's just a warning message. It shouldn't cause the file load to > fail. Derek and David, how it got in? no idea, I don't think I entered it manually, so it probably, got in through the price editor. I would assume that it has been in for some time. David states, that it should not abort on reading this date, but changing it to 1971-01-01 12:00:00 +0100 fixes it: no abort any more. odd... Eildert -- Eildert Groeneveld === Institute for Animal Breeding Mariensee 31535 Neustadt Germany Tel : (+49)(0)5034 871155 Fax : (+49)(0)5034 871239 e-mail: [EMAIL PROTECTED] http://vce.tzv.fal.de/index.pl http://www.tzv.fal.de/~eg/ http://apiis.tzv.fal.de/index.pl === ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: At revision 13129: error reading gnucash file
On Mon, 2006-02-06 at 19:36 +0100, Eildert Groeneveld wrote: > On Monday 06 February 2006 17:53, Derek Atkins wrote: > > Anything else printed on the terminal? > > What about in /tmp/gnucash.trace? > alright, this is what I see there: > > > loading stylesheets > looking for /home2/eg/.gnucash/stylesheets-2.0 > trying to load /home2/eg/.gnucash/stylesheets-2.0 > Invalid timestamp in data file. Look for a 'price:time' entry with a date of > 1969-12-31 or 1970-01-01. > > for loglevel 2 > > does this help? That's just a warning message. It shouldn't cause the file load to fail. David ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: At revision 13129: error reading gnucash file
Quoting Eildert Groeneveld <[EMAIL PROTECTED]>: On Monday 06 February 2006 20:03, Derek Atkins wrote: I believe that yes, that is an illegal date.. Or at least it is a "confusing" date. It's a date of "0", which can certainly be confusing. You should probably remove that price from the database, or change the date to be less "confusing". Perhaps 1971-01-01 12:00:00 +0100? Derek that did the trick. Thanx! Perhaps we should have this caught in the code? Well, how did this particular entry get in the database in the first place? greetings -derek -- Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory Member, MIT Student Information Processing Board (SIPB) URL: http://web.mit.edu/warlord/PP-ASEL-IA N1NWH [EMAIL PROTECTED]PGP key available ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: At revision 13129: error reading gnucash file
On Monday 06 February 2006 20:03, Derek Atkins wrote: > I believe that yes, that is an illegal date.. Or at least it is a > "confusing" date. It's a date of "0", which can certainly be confusing. > > You should probably remove that price from the database, or change the > date to be less "confusing". Perhaps 1971-01-01 12:00:00 +0100? Derek that did the trick. Thanx! Perhaps we should have this caught in the code? greetings Eildert ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: At revision 13129: error reading gnucash file
I believe that yes, that is an illegal date.. Or at least it is a "confusing" date. It's a date of "0", which can certainly be confusing. You should probably remove that price from the database, or change the date to be less "confusing". Perhaps 1971-01-01 12:00:00 +0100? -derek Quoting Eildert Groeneveld <[EMAIL PROTECTED]>: On Monday 06 February 2006 19:48, Derek Atkins wrote: Did you do what the error suggested? well, there is a line in the gnucash file: Info: dom_tree_to_text(): node string is >>>EUR<<< Info: dom_tree_to_text(): node string is >>>1970-01-01 01:00:00 +0100<<< but is this an illegal date? what can one do? Eildert -derek Quoting Eildert Groeneveld <[EMAIL PROTECTED]>: > On Monday 06 February 2006 17:53, Derek Atkins wrote: >> Anything else printed on the terminal? >> What about in /tmp/gnucash.trace? > > alright, this is what I see there: > > > loading stylesheets > looking for /home2/eg/.gnucash/stylesheets-2.0 > trying to load /home2/eg/.gnucash/stylesheets-2.0 > Invalid timestamp in data file. Look for a 'price:time' entry with a > date of 1969-12-31 or 1970-01-01. > > for loglevel 2 > > does this help? > > Eilder > >> -derek >> >> Quoting Eildert Groeneveld <[EMAIL PROTECTED]>: >> > Hello folks >> > >> > At revision 13129 gnucash aborts on reading the gnucash file: >> > >> > an error occurred when reading file soandso. >> > >> > Is this also happening somewhere else? I have tried a few backup >> > files; none of them works. >> > >> > setting highest debug levele I get: >> > >> > File size limit exceeded >> > >> > any suggestions? >> > >> > greetings >> > >> > Eildert >> > >> > >> > >> > -- >> > Eildert Groeneveld >> > ___ >> > gnucash-devel mailing list >> > gnucash-devel@gnucash.org >> > https://lists.gnucash.org/mailman/listinfo/gnucash-devel > > -- > Eildert Groeneveld > === > Institute for Animal Breeding > Mariensee 31535 Neustadt Germany > Tel : (+49)(0)5034 871155 Fax : (+49)(0)5034 871239 > e-mail: [EMAIL PROTECTED] http://vce.tzv.fal.de/index.pl > http://www.tzv.fal.de/~eg/ http://apiis.tzv.fal.de/index.pl > === -- Eildert Groeneveld === Institute for Animal Breeding Mariensee 31535 Neustadt Germany Tel : (+49)(0)5034 871155 Fax : (+49)(0)5034 871239 e-mail: [EMAIL PROTECTED] http://vce.tzv.fal.de/index.pl http://www.tzv.fal.de/~eg/ http://apiis.tzv.fal.de/index.pl === -- Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory Member, MIT Student Information Processing Board (SIPB) URL: http://web.mit.edu/warlord/PP-ASEL-IA N1NWH [EMAIL PROTECTED]PGP key available ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: At revision 13129: error reading gnucash file
On Monday 06 February 2006 19:48, Derek Atkins wrote: > Did you do what the error suggested? well, there is a line in the gnucash file: Info: dom_tree_to_text(): node string is >>>EUR<<< Info: dom_tree_to_text(): node string is >>>1970-01-01 01:00:00 +0100<<< but is this an illegal date? what can one do? Eildert > > -derek > > Quoting Eildert Groeneveld <[EMAIL PROTECTED]>: > > On Monday 06 February 2006 17:53, Derek Atkins wrote: > >> Anything else printed on the terminal? > >> What about in /tmp/gnucash.trace? > > > > alright, this is what I see there: > > > > > > loading stylesheets > > looking for /home2/eg/.gnucash/stylesheets-2.0 > > trying to load /home2/eg/.gnucash/stylesheets-2.0 > > Invalid timestamp in data file. Look for a 'price:time' entry with a > > date of 1969-12-31 or 1970-01-01. > > > > for loglevel 2 > > > > does this help? > > > > Eilder > > > >> -derek > >> > >> Quoting Eildert Groeneveld <[EMAIL PROTECTED]>: > >> > Hello folks > >> > > >> > At revision 13129 gnucash aborts on reading the gnucash file: > >> > > >> > an error occurred when reading file soandso. > >> > > >> > Is this also happening somewhere else? I have tried a few backup > >> > files; none of them works. > >> > > >> > setting highest debug levele I get: > >> > > >> > File size limit exceeded > >> > > >> > any suggestions? > >> > > >> > greetings > >> > > >> > Eildert > >> > > >> > > >> > > >> > -- > >> > Eildert Groeneveld > >> > ___ > >> > gnucash-devel mailing list > >> > gnucash-devel@gnucash.org > >> > https://lists.gnucash.org/mailman/listinfo/gnucash-devel > > > > -- > > Eildert Groeneveld > > === > > Institute for Animal Breeding > > Mariensee 31535 Neustadt Germany > > Tel : (+49)(0)5034 871155 Fax : (+49)(0)5034 871239 > > e-mail: [EMAIL PROTECTED] http://vce.tzv.fal.de/index.pl > > http://www.tzv.fal.de/~eg/ http://apiis.tzv.fal.de/index.pl > > === -- Eildert Groeneveld === Institute for Animal Breeding Mariensee 31535 Neustadt Germany Tel : (+49)(0)5034 871155 Fax : (+49)(0)5034 871239 e-mail: [EMAIL PROTECTED] http://vce.tzv.fal.de/index.pl http://www.tzv.fal.de/~eg/ http://apiis.tzv.fal.de/index.pl === ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: [Gnucash-changes] r13132 - gnucash/trunk/src/engine - Keep QOF aware of the dirty-state of Transactions.
Quoting Chris Shoemaker <[EMAIL PROTECTED]>: mark_trans() is horribly named. It's a fundamentally different operation than qof_instance_set_dirty(). _Most_ of the places we call qof_instance_dirty (11 of them) we don't really want to dirty the splits. I admit I haven't looked at the code recently.. Part of the issue is that in XML the Splits aren't listed on their own -- they only exist within a Transaction -- so you kinda need to mark the transaction as dirty when you change a split. I'm not sure in what cases you ever want to or need to mark a split dirty just because the transaction changed. On a separate (but related) note, I want to separate extend the events in order to differentiate between "Account data modified" and "account contents modified" -- where the former is emitted by the new/edit account dialog, and the latter is emitted by the register... Absolutely, and this is _very_ related, because the same case applies to Transactions. The mark_trans() cases should correlate with emitting the "child_changed" signal (or whatever), while the qof_instance_set_dirty() cases should correlate with emitting just the "changed" signal (or whatever). And in two places, we should emit both. We need to come up with a good set of signals here. "MODIFIED" isn't sufficiently robust. :) Unfortunately "INSERTED" and "REMOVED" are already used, and used IMHO in weird ways. :( This is a very useful distinction for both Accounts and Transactions. ... and Invoices. Indeed, another question is who owns the link. For example, I think the Split<->Account mapping is backwards.. Right now you need to Begin/Commit on the Account, but it's really the Split that contains the mapping. The XML (and SQL) data for an account does not keep track of the splits. The split keep track of the account. So, shouldn't it be xaccSplitSetAccount() instead of xaccAccountInsertSplit(), and shouldn't that be wrapped around a Split/Trans Begin/Commit edit? (I'll note that we still need the xaccAccountInsertSplit(), but I think that only should be called "interally" by xaccSplitSetAccount(), and it doesn't necessarily need a begin/commit edit because it's only structural data, not real data). -derek -- Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory Member, MIT Student Information Processing Board (SIPB) URL: http://web.mit.edu/warlord/PP-ASEL-IA N1NWH [EMAIL PROTECTED]PGP key available ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: At revision 13129: error reading gnucash file
Did you do what the error suggested? -derek Quoting Eildert Groeneveld <[EMAIL PROTECTED]>: On Monday 06 February 2006 17:53, Derek Atkins wrote: Anything else printed on the terminal? What about in /tmp/gnucash.trace? alright, this is what I see there: loading stylesheets looking for /home2/eg/.gnucash/stylesheets-2.0 trying to load /home2/eg/.gnucash/stylesheets-2.0 Invalid timestamp in data file. Look for a 'price:time' entry with a date of 1969-12-31 or 1970-01-01. for loglevel 2 does this help? Eilder -derek Quoting Eildert Groeneveld <[EMAIL PROTECTED]>: > Hello folks > > At revision 13129 gnucash aborts on reading the gnucash file: > > an error occurred when reading file soandso. > > Is this also happening somewhere else? I have tried a few backup files; > none of them works. > > setting highest debug levele I get: > > File size limit exceeded > > any suggestions? > > greetings > > Eildert > > > > -- > Eildert Groeneveld > ___ > gnucash-devel mailing list > gnucash-devel@gnucash.org > https://lists.gnucash.org/mailman/listinfo/gnucash-devel -- Eildert Groeneveld === Institute for Animal Breeding Mariensee 31535 Neustadt Germany Tel : (+49)(0)5034 871155 Fax : (+49)(0)5034 871239 e-mail: [EMAIL PROTECTED] http://vce.tzv.fal.de/index.pl http://www.tzv.fal.de/~eg/ http://apiis.tzv.fal.de/index.pl === -- Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory Member, MIT Student Information Processing Board (SIPB) URL: http://web.mit.edu/warlord/PP-ASEL-IA N1NWH [EMAIL PROTECTED]PGP key available ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: At revision 13129: error reading gnucash file
On Monday 06 February 2006 17:53, Derek Atkins wrote: > Anything else printed on the terminal? > What about in /tmp/gnucash.trace? alright, this is what I see there: loading stylesheets looking for /home2/eg/.gnucash/stylesheets-2.0 trying to load /home2/eg/.gnucash/stylesheets-2.0 Invalid timestamp in data file. Look for a 'price:time' entry with a date of 1969-12-31 or 1970-01-01. for loglevel 2 does this help? Eilder > > -derek > > Quoting Eildert Groeneveld <[EMAIL PROTECTED]>: > > Hello folks > > > > At revision 13129 gnucash aborts on reading the gnucash file: > > > > an error occurred when reading file soandso. > > > > Is this also happening somewhere else? I have tried a few backup files; > > none of them works. > > > > setting highest debug levele I get: > > > > File size limit exceeded > > > > any suggestions? > > > > greetings > > > > Eildert > > > > > > > > -- > > Eildert Groeneveld > > ___ > > gnucash-devel mailing list > > gnucash-devel@gnucash.org > > https://lists.gnucash.org/mailman/listinfo/gnucash-devel -- Eildert Groeneveld === Institute for Animal Breeding Mariensee 31535 Neustadt Germany Tel : (+49)(0)5034 871155 Fax : (+49)(0)5034 871239 e-mail: [EMAIL PROTECTED] http://vce.tzv.fal.de/index.pl http://www.tzv.fal.de/~eg/ http://apiis.tzv.fal.de/index.pl === ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: [Gnucash-changes] r13132 - gnucash/trunk/src/engine - Keep QOF aware of the dirty-state of Transactions.
On Mon, Feb 06, 2006 at 01:11:31PM -0500, Derek Atkins wrote: > Quoting Chris Shoemaker <[EMAIL PROTECTED]>: > > >On Mon, Feb 06, 2006 at 01:03:43PM -0500, Derek Atkins wrote: > >>mark_trans() should call qof_instance_set_dirty(). no need to call it > >>everywhere -- we already call mark_trans() everywhere. Do this all in > >>one place, please. > > > >Actually, we call mark_trans() in only two places. It seems like > >mark_trans() is for dirtying the splits in the trans, not the trans > >itself. > > Hmm.. I'm thinking we might want to consolidate some. There's lots of > places where we want to mark the transaction dirty, notify QOF, and send > an event. This should all be done in one function and that function > should be called everywhere. mark_trans() is horribly named. It's a fundamentally different operation than qof_instance_set_dirty(). _Most_ of the places we call qof_instance_dirty (11 of them) we don't really want to dirty the splits. > On a separate (but related) note, I want to separate extend the events > in order to differentiate between "Account data modified" and "account > contents modified" -- where the former is emitted by the new/edit account > dialog, and the latter is emitted by the register... Absolutely, and this is _very_ related, because the same case applies to Transactions. The mark_trans() cases should correlate with emitting the "child_changed" signal (or whatever), while the qof_instance_set_dirty() cases should correlate with emitting just the "changed" signal (or whatever). And in two places, we should emit both. This is a very useful distinction for both Accounts and Transactions. -chris > I just haven't > gone through to handle this yet -- I'm going to have to go through the > code and find every place the account events are used and check them. > Blah. > > -derek > > -- > Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory > Member, MIT Student Information Processing Board (SIPB) > URL: http://web.mit.edu/warlord/PP-ASEL-IA N1NWH > [EMAIL PROTECTED]PGP key available > ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: [Gnucash-changes] r13132 - gnucash/trunk/src/engine - Keep QOF aware of the dirty-state of Transactions.
Quoting Chris Shoemaker <[EMAIL PROTECTED]>: On Mon, Feb 06, 2006 at 01:03:43PM -0500, Derek Atkins wrote: mark_trans() should call qof_instance_set_dirty(). no need to call it everywhere -- we already call mark_trans() everywhere. Do this all in one place, please. Actually, we call mark_trans() in only two places. It seems like mark_trans() is for dirtying the splits in the trans, not the trans itself. Hmm.. I'm thinking we might want to consolidate some. There's lots of places where we want to mark the transaction dirty, notify QOF, and send an event. This should all be done in one function and that function should be called everywhere. On a separate (but related) note, I want to separate extend the events in order to differentiate between "Account data modified" and "account contents modified" -- where the former is emitted by the new/edit account dialog, and the latter is emitted by the register... I just haven't gone through to handle this yet -- I'm going to have to go through the code and find every place the account events are used and check them. Blah. -derek -- Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory Member, MIT Student Information Processing Board (SIPB) URL: http://web.mit.edu/warlord/PP-ASEL-IA N1NWH [EMAIL PROTECTED]PGP key available ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: [Gnucash-changes] r13132 - gnucash/trunk/src/engine - Keep QOF aware of the dirty-state of Transactions.
On Mon, Feb 06, 2006 at 01:03:43PM -0500, Derek Atkins wrote: > mark_trans() should call qof_instance_set_dirty(). no need to call it > everywhere -- we already call mark_trans() everywhere. Do this all in > one place, please. Actually, we call mark_trans() in only two places. It seems like mark_trans() is for dirtying the splits in the trans, not the trans itself. -chris > > -derek > > Quoting Chris Shoemaker <[EMAIL PROTECTED]>: > > >Modified: > > gnucash/trunk/src/engine/Transaction.c > >Log: > > Keep QOF aware of the dirty-state of Transactions. > > Also, dirty transactions upon changing due date and read-only status, > > which didn't used to happen. > [snip] > >+ qof_instance_set_dirty(QOF_INSTANCE(trans)); > > mark_trans (trans); > > qof_commit_edit(QOF_INSTANCE(trans)); > [snip] > > -- > Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory > Member, MIT Student Information Processing Board (SIPB) > URL: http://web.mit.edu/warlord/PP-ASEL-IA N1NWH > [EMAIL PROTECTED]PGP key available > ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: [Gnucash-changes] r13132 - gnucash/trunk/src/engine - Keep QOF aware of the dirty-state of Transactions.
mark_trans() should call qof_instance_set_dirty(). no need to call it everywhere -- we already call mark_trans() everywhere. Do this all in one place, please. -derek Quoting Chris Shoemaker <[EMAIL PROTECTED]>: Modified: gnucash/trunk/src/engine/Transaction.c Log: Keep QOF aware of the dirty-state of Transactions. Also, dirty transactions upon changing due date and read-only status, which didn't used to happen. [snip] + qof_instance_set_dirty(QOF_INSTANCE(trans)); mark_trans (trans); qof_commit_edit(QOF_INSTANCE(trans)); [snip] -- Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory Member, MIT Student Information Processing Board (SIPB) URL: http://web.mit.edu/warlord/PP-ASEL-IA N1NWH [EMAIL PROTECTED]PGP key available ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: [Gnucash-changes] r13131 - gnucash/trunk/src - Convert Split from QofEntity to QofInstance.
On Mon, Feb 06, 2006 at 12:52:13PM -0500, Derek Atkins wrote: > With this changeset the build breaks: > > io-gncxml-v1.c: In function 'txn_restore_split_after_child_handler': > io-gncxml-v1.c:3115: error: 'struct split_s' has no member named 'kvp_data' > io-gncxml-v1.c:3115: error: 'struct split_s' has no member named 'kvp_data' > io-gncxml-v1.c:3116: error: 'struct split_s' has no member named 'kvp_data' > make[5]: *** [io-gncxml-v1.lo] Error 1 > make[5]: Leaving directory > `/home/warlord/src/gnucash/gnucash-svn/build/src/backend/file' Oops. I wonder why my compile check picked up the breakage in backend/postgres, but not backend/file. I'll fix it momentarily. -chris > > -derek > > Quoting Chris Shoemaker <[EMAIL PROTECTED]>: > > >Modified: > > gnucash/trunk/src/backend/postgres/txn.c > > gnucash/trunk/src/engine/Account.c > > gnucash/trunk/src/engine/Period.c > > gnucash/trunk/src/engine/Scrub2.c > > gnucash/trunk/src/engine/Transaction.c > > gnucash/trunk/src/engine/TransactionP.h > > gnucash/trunk/src/engine/cap-gains.c > >Log: > > Convert Split from QofEntity to QofInstance. > > Let QofInstance manage the book pointer and kvp frame instead of Split > > managing its own. > > -- > Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory > Member, MIT Student Information Processing Board (SIPB) > URL: http://web.mit.edu/warlord/PP-ASEL-IA N1NWH > [EMAIL PROTECTED]PGP key available > ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: [Gnucash-changes] r13131 - gnucash/trunk/src - Convert Split from QofEntity to QofInstance.
With this changeset the build breaks: io-gncxml-v1.c: In function 'txn_restore_split_after_child_handler': io-gncxml-v1.c:3115: error: 'struct split_s' has no member named 'kvp_data' io-gncxml-v1.c:3115: error: 'struct split_s' has no member named 'kvp_data' io-gncxml-v1.c:3116: error: 'struct split_s' has no member named 'kvp_data' make[5]: *** [io-gncxml-v1.lo] Error 1 make[5]: Leaving directory `/home/warlord/src/gnucash/gnucash-svn/build/src/backend/file' -derek Quoting Chris Shoemaker <[EMAIL PROTECTED]>: Modified: gnucash/trunk/src/backend/postgres/txn.c gnucash/trunk/src/engine/Account.c gnucash/trunk/src/engine/Period.c gnucash/trunk/src/engine/Scrub2.c gnucash/trunk/src/engine/Transaction.c gnucash/trunk/src/engine/TransactionP.h gnucash/trunk/src/engine/cap-gains.c Log: Convert Split from QofEntity to QofInstance. Let QofInstance manage the book pointer and kvp frame instead of Split managing its own. -- Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory Member, MIT Student Information Processing Board (SIPB) URL: http://web.mit.edu/warlord/PP-ASEL-IA N1NWH [EMAIL PROTECTED]PGP key available ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: Fancy Invoice
Quoting Andrew Sackville-West <[EMAIL PROTECTED]>: So what is the proper way to look up that specific info from the book database? Well, I think the phone number and website would need to be added to the book properties (take a look in business-prefs.scm). Then you can load the book properties and lookup the contents (see invoice.scm for an example of how it pulls in the company name and address). A -derek -- Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory Member, MIT Student Information Processing Board (SIPB) URL: http://web.mit.edu/warlord/PP-ASEL-IA N1NWH [EMAIL PROTECTED]PGP key available ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: Fancy Invoice
On Mon, 6 Feb 2006 11:51:08 -0500 Derek Atkins <[EMAIL PROTECTED]> wrote: > Quoting Neil Williams <[EMAIL PROTECTED]>: > > > Where is the report looking to find the Phone and website details? > > (Please don't tell me I have to edit the scheme!) > > :-) > > You have to edit the scheme. The Fancy Invoice was a donated report, > and the donator did not use the appropriate methods to lookup the > company name or phone number in the book database. So, he just hard > coded those strings into the scheme. Annoying, yes. Nobody has > submitted a patch to fix it, and I haven't taken the time to do it > because I don't use the Fancy Invoice. So what is the proper way to look up that specific info from the book database? A > > So, I'm sorry to say you have to edit the scheme. > > -derek > -- >Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory >Member, MIT Student Information Processing Board (SIPB) >URL: http://web.mit.edu/warlord/PP-ASEL-IA N1NWH >[EMAIL PROTECTED]PGP key available > > ___ > gnucash-devel mailing list > gnucash-devel@gnucash.org > https://lists.gnucash.org/mailman/listinfo/gnucash-devel pgptQEmPAxhzo.pgp Description: PGP signature ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: At revision 13129: error reading gnucash file
Anything else printed on the terminal? What about in /tmp/gnucash.trace? -derek Quoting Eildert Groeneveld <[EMAIL PROTECTED]>: Hello folks At revision 13129 gnucash aborts on reading the gnucash file: an error occurred when reading file soandso. Is this also happening somewhere else? I have tried a few backup files; none of them works. setting highest debug levele I get: File size limit exceeded any suggestions? greetings Eildert -- Eildert Groeneveld ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel -- Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory Member, MIT Student Information Processing Board (SIPB) URL: http://web.mit.edu/warlord/PP-ASEL-IA N1NWH [EMAIL PROTECTED]PGP key available ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: Fancy Invoice
Quoting Neil Williams <[EMAIL PROTECTED]>: Where is the report looking to find the Phone and website details? (Please don't tell me I have to edit the scheme!) :-) You have to edit the scheme. The Fancy Invoice was a donated report, and the donator did not use the appropriate methods to lookup the company name or phone number in the book database. So, he just hard coded those strings into the scheme. Annoying, yes. Nobody has submitted a patch to fix it, and I haven't taken the time to do it because I don't use the Fancy Invoice. So, I'm sorry to say you have to edit the scheme. -derek -- Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory Member, MIT Student Information Processing Board (SIPB) URL: http://web.mit.edu/warlord/PP-ASEL-IA N1NWH [EMAIL PROTECTED]PGP key available ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Fancy Invoice
When viewing the Fancy Invoice report, I get: Phone: (111) 222- Web: http://companysite.com I've found how to change the lines: Make all cheques payable to: and Direct all enquiries to: in the report options -> Display tab -> Extra Notes. Where is the report looking to find the Phone and website details? (Please don't tell me I have to edit the scheme!) :-) -- Neil Williams = http://www.data-freedom.org/ http://www.nosoftwarepatents.com/ http://www.linux.codehelp.co.uk/ pgpP4ma9IZ6vX.pgp Description: PGP signature ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
At revision 13129: error reading gnucash file
Hello folks At revision 13129 gnucash aborts on reading the gnucash file: an error occurred when reading file soandso. Is this also happening somewhere else? I have tried a few backup files; none of them works. setting highest debug levele I get: File size limit exceeded any suggestions? greetings Eildert -- Eildert Groeneveld ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: Problem with r13089
Chris Lyttle <[EMAIL PROTECTED]> writes: > FAILURE > engine-stuff ../../../../src/engine/test-core/test-engine-stuff.c:1451 > get_random_transaction failed > FAILURE > transaction_xml > ../../../../../src/backend/file/test/test-xml-transaction.c:379 > get_random_transaction returned NULL This is a "bad luck" test. If you have really bad luck and the RNG gives you an account tree with only one account, it's possible to see this error. I've fixed it in SVN to make sure you always have at least two accounts. -derek -- Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory Member, MIT Student Information Processing Board (SIPB) URL: http://web.mit.edu/warlord/PP-ASEL-IA N1NWH [EMAIL PROTECTED]PGP key available ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
Re: problems with g2
Can you update to 13129 and try again? If you can still do this then please file a bug report in bugzilla. I know that there were fixes for the gconf settings. I don't know about the File Save-As. -derek Mark Johnson <[EMAIL PROTECTED]> writes: > Hi, > > I built and installed g2 by: > make distclean > svn co http:/svn.gnucash.org/repo/gnucash/trunk > ./autogen.sh > ./configure --enable-opt-style-install --prefix=/opt/gnucash-svn > make > su > make install > > The revision obtained was 12327. > > I changed to the directory /opt/gnucash-svn/bin and ran gnucash from > there (the directory is not on my PATH). When I started it, it put up a > druid for some configuration. I did not note the specifics of this, but > it was something to do with a path in .gconf. > > I was able to create a few accounts, but when I tried to enter > transactions, the numbers would disappear when I left the cells by > either tabbing or pressing enter. > > When I selected "File-Save As", it allowed me to create a new directory > in my home, I entered a file name and clicked OK. I got an error stating > that it was unable to parse the URL; the URL being the path and file > name to which I had selected saving the data. It had nothing like > "file://" on the beginning. It was simply > "/mnt/raid/home/mj/gnucash-svn12327/books.xac". > > Both these issues seem too obvious not to have been noticed. Any ideas? > > In 1.8.12, when selecting an account for a split in the register, > pressing ":" would complete the account name and allow one to select an > account lower in the hierarchy. This no longer worked. One really > notices the lack of convenience when it is gone. > > Mark > ___ > gnucash-devel mailing list > gnucash-devel@gnucash.org > https://lists.gnucash.org/mailman/listinfo/gnucash-devel > > -- Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory Member, MIT Student Information Processing Board (SIPB) URL: http://web.mit.edu/warlord/PP-ASEL-IA N1NWH [EMAIL PROTECTED]PGP key available ___ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel