SLR bug in 2.2?

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

SLR bug in 2.2?

Tim Wunder (Lists)
In 2.0.x, if the Since Last Run druid is canceled at any time, all
transactions are canceled. In 2.2 it seems that if you cancel the SLR after
reviewing the created transactions, the auto created transactions are not
canceled (and the book gets dirtied). Is this intended behavior? It strikes
me as a bug. Cancel from the SLR, be it at the initial screen or during
review, should cancel all SXs.

Tim
--
Fedora Core release 6 (Zod), Linux 2.6.22.2-42.fc6
KDE: 3.5.7-9.fc6 Fedora
 23:35:01 up  8:39,  3 users,  load average: 0.00, 0.00, 0.01
"It's what you learn after you know it all that counts" John Wooden

_______________________________________________
gnucash-devel mailing list
[hidden email]
https://lists.gnucash.org/mailman/listinfo/gnucash-devel

signature.asc (196 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: SLR bug in 2.2?

Josh Sled
Tim Wunder <[hidden email]> writes:
> In 2.0.x, if the Since Last Run druid is canceled at any time, all
> transactions are canceled. In 2.2 it seems that if you cancel the SLR after
> reviewing the created transactions, the auto created transactions are not
> canceled (and the book gets dirtied). Is this intended behavior? It strikes
> me as a bug. Cancel from the SLR, be it at the initial screen or during
> review, should cancel all SXs.

It is the intended behavior.  There is no longer a review step as part of the
multi-step druid, but instead a single post-SLR way to review (and modify)
which transactions were created.

In truth, that review page also dirtied the book, while breaking conventions
about how the "Cancel" operation on a Druid should function, as it needed to
actually create the transactions in order to "review" them, there.  I
considered that the bigger failing.  Hopefully the register-rerewrite branch
will make it easier to have a register of not-actually-created transactions,
which would make it possible to restore the "review" functionality.

--
...jsled
http://asynchronous.org/ - a=jsled; b=asynchronous.org; echo ${a}@${b}

_______________________________________________
gnucash-devel mailing list
[hidden email]
https://lists.gnucash.org/mailman/listinfo/gnucash-devel

attachment0 (194 bytes) Download Attachment