[Koha-bugs] [Bug 24820] The cashup workflow should use the static 'date' field rather than the transient 'timestamp' field in accountlines

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24820

--- Comment #15 from Katrin Fischer <katrin.fischer at bsz-bw.de> ---
(In reply to Martin Renvoize from comment #13)
> (In reply to Katrin Fischer from comment #12)
> > Ok, it might be late, but why only change one timestamp to date?
> > 
> > -      ? { 'timestamp' => { '>' => $since->get_column('timestamp')->as_query
> > } }
> > +      ? { 'date' => { '>' => $since->get_column('timestamp')->as_query } }
> 
> Sorry Katrin, not sure I understand your question there?
> 
> 'date' is the field in the accountlines table and 'timestamp' is the field
> in the cash_register_actions table.
> 
> Are you suggesting I update the cash_register_actions table to have a field
> name of 'date' to mate the accountlines 'date' field name?  If we're
> thinking of doing that I'd actually prefer to change both tables to
> 'created' or similar to denote exactly their use but I felt that's outside
> of scope for this bug

Just me being confused - it didn't occur to me we were looking at values from 2
different tables here...

-- 
You are receiving this mail because:
You are watching all bug changes.