I am in favor of supporting stateless Activities. There are too many good use cases for them to not support having them. The Log Activity is a good example. There is no much point in having a Journal entry for Log. Old Journal entries do not point to old logs. (If they did there might be some point to having them, I suppose). Games might do better to save state information in the data directory rather than create a Journal entry, if the state is nothing more than high scores.<br>
<br>The parameter in the __init__() method looks like it has been there from the beginning of Sugar.<br><br>I don't mind changing the book if that is what needs to be done, but I think the trick documented in the book is worth preserving.<br>
<br>James Simmons<br><br><br><div class="gmail_quote">On Thu, Aug 4, 2011 at 1:07 AM, James Cameron <span dir="ltr"><<a href="mailto:quozl@laptop.org">quozl@laptop.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">On Thu, Aug 04, 2011 at 10:27:09AM +0500, Sebastian Silva wrote:<br>
> I just tried Log activity, it does create a Journal entry by default.<br>
<br>
</div>This has since changed. As I said, look at the Log activity source ...<br>
sorry I wasn't explicit enough ... in the git repository.<br>
<br>
Specifically,<br>
<a href="http://git.sugarlabs.org/log/mainline/blobs/master/logviewer.py#line329" target="_blank">http://git.sugarlabs.org/log/mainline/blobs/master/logviewer.py#line329</a><br>
is where the activity instructs Sugar not to create a Journal entry by<br>
default.<br>
<br>
And,<br>
<a href="http://git.sugarlabs.org/log/mainline/blobs/master/logviewer.py#line511" target="_blank">http://git.sugarlabs.org/log/mainline/blobs/master/logviewer.py#line511</a><br>
is where a Journal entry is created as a result of user action.<br>
<br>
Sorry nobody has released a new version of Log yet. That might be a<br>
good idea.<br>
<div class="im"><br>
> So I wonder, is the workaround described in the book supported?<br>
<br>
</div>Well, I don't know about supported, but it clearly doesn't work. Let's<br>
find out why and get it fixed, or changed. That is, in Sugar or the<br>
book. Books are code too, but they are slightly harder to change.<br>
<font color="#888888"><br>
--<br>
James Cameron<br>
<a href="http://quozl.linux.org.au/" target="_blank">http://quozl.linux.org.au/</a><br>
_______________________________________________<br>
Sugar-devel mailing list<br>
<a href="mailto:Sugar-devel@lists.sugarlabs.org">Sugar-devel@lists.sugarlabs.org</a><br>
<a href="http://lists.sugarlabs.org/listinfo/sugar-devel" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
</font></blockquote></div><br>