Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#4352 closed defect (fixed)

Language control panel lets you delete first langauge

Reported by: greenfeld Owned by: erikos
Priority: Unspecified by Maintainer Milestone:
Component: Sugar Version: 0.98.x
Severity: Major Keywords: regression, r+
Cc: Distribution/OS: OLPC
Bug Status: Unconfirmed

Description

The Language control panel in 13.1.0 os11 lets you delete the first language choice, leaving no language dropdown in the control panel.

It is possible to get in a state where the control panel asks you to restart Sugar after this; but usually it does not, and so far I have not managed to save a blank ~/.i18n file.

Seen in 13.1.0 os20 (along with #4327 & #4329).

Attachments (1)

0005-Hide-the-Remove-button-if-there-is-only-one-language.patch (1.9 KB) - added by manuq 8 years ago.
Patch.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 8 years ago by manuq

Traceback:

Traceback (most recent call last):
  File "/home/manuq/sugar-build/install/share/sugar/extensions/cpsection/languag
    self._delete_last_row()
  File "/home/manuq/sugar-build/install/share/sugar/extensions/cpsection/languag
    self._add_remove_boxes[-1].show_all()
IndexError: list index out of range

comment:2 Changed 8 years ago by manuq

  • Keywords r? added

comment:3 Changed 8 years ago by erikos

  • Keywords r+ added; r? removed
  • Resolution set to fixed
  • Status changed from new to closed

Cool, thanks for the patch: master=0a98409be5eedb9ee27a9fd1b526e99c764f55f5 0.98=c48dcb87493d21528f26013432e0d952fd303686

Looks good, and tested as working.

comment:4 Changed 8 years ago by dnarvaez

  • Milestone 0.98 deleted

Milestone 0.98 deleted

Note: See TracTickets for help on using tickets.