Opened 10 years ago

Closed 10 years ago

#2441 closed enhancement (fixed)

Multiple turtles inconsistent behaviour with clean

Reported by: tonyforster Owned by: walter
Priority: Unspecified by Maintainer Milestone: Unspecified
Component: Turtleart Version: Unspecified
Severity: Unspecified Keywords:
Cc: Distribution/OS: Unspecified
Bug Status: Unconfirmed

Description

Clean erases multiple turtles but they return if you rename the activity, go to journal and back or quit.
OS17, 0.90 XO1 V99

Attachments (1)

Screenshot3turtles.png (44.5 KB) - added by tonyforster 10 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 follow-up: Changed 10 years ago by walter

  • Type changed from defect to enhancement

Clean doesn't erase the turtles... it just brings them all back to 'home'. You can erase turtles by dragging them on to the palette.

Should clean erase the turtles?

comment:2 Changed 10 years ago by walter

Could you please do your further testing with the new release candidate, v100, available here:

http://wiki.sugarlabs.org/go/File:TurtleArt-100.xo

comment:3 in reply to: ↑ 1 Changed 10 years ago by tonyforster

Replying to walter:

Clean doesn't erase the turtles... it just brings them all back to 'home'. You can erase turtles by dragging them on to the palette.

Should clean erase the turtles?

Thanks. V99 clean also hides the turtles. The user would expect them to stay hidden. My preference is that clean erase (or hide) them but what is important is that whatever clean does should be consistent.

Will try on V100.

comment:4 Changed 10 years ago by tonyforster

The attached screenshot illustrates the problem, the oscilloscope has been used on 3 machines and the ta file has picked up 3 turtles, 2 unknown to the user

["turtle", "Tony"], 0, 0, 0, 0, 50, 5], [-1, ["turtle", "Tony1.5"], -316, 113, 0, 0, 50, 5], [-1, ["turtle", "Yertle"], 380, 113, 0, 0, 50, 5]]

As the screenshot shows, alt tab unhides the 2 turtles

Changed 10 years ago by tonyforster

comment:5 Changed 10 years ago by walter

  • Status changed from new to assigned

Maybe rather than saving the default turtle by the user's nick, I should just save it as 'Yertle', which is the default; then I can test for that case and not inadvertently load or save extra turtles.

#2687 includes a patch to prevent shared turtles from getting saved with a project.

comment:6 Changed 10 years ago by walter

  • Resolution set to fixed
  • Status changed from assigned to closed

In theory this patch (with the wrong commit message) will fix the problem above (but not retroactively).

http://git.sugarlabs.org/turtleart/mainline/commit/b658b9f0c3856fe496c8bc9707aaab512b3ae829

Note: See TracTickets for help on using tickets.