[bug #52494] Does not handle exceptions when loading translations

[Yavor Doganov]

Follow-up Comment #3, bug #52494 (project gnustep):

The applied fix is basically my proposed patch, with one minor nuance.  I have
already tested it.

___

Reply to this item at:

  

___
  Message sent via/by Savannah
  http://savannah.gnu.org/


___
Bug-gnustep mailing list
Bug-gnustep@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-gnustep

[bug #52494] Does not handle exceptions when loading translations

[Gregory John Casamento]

Update of bug #52494 (project gnustep):

  Status:None => Fixed  
 Assigned to:None => gcasa  
 Open/Closed:Open => In Test

___

Follow-up Comment #2:

This should now be fixed.  Please test.

___

Reply to this item at:

  

___
  Message sent via/by Savannah
  http://savannah.gnu.org/


___
Bug-gnustep mailing list
Bug-gnustep@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-gnustep

[bug #52494] Does not handle exceptions when loading translations

[Yavor Doganov]

Follow-up Comment #1, bug #52494 (project gnustep):

One thing is bothering me... GUI sets its own exception handler so if
application code does not catch an exception, a critical alert panel must be
shown in addition to the usual logging that comes from Base.   In this case
there is no alert panel and nothing is logged.  I thought it might be a bug in
the runtime but at first glance it doesn't seem so.  A minimalistic program
([NSApplication sharedApplication] immediately followed by code that triggers
an exception) works as expected.

I'm quite certain there's another bug here; uncaught exceptions shouldn't be
swept under the carpet.  Any ideas?

___

Reply to this item at:

  

___
  Message sent via/by Savannah
  http://savannah.gnu.org/


___
Bug-gnustep mailing list
Bug-gnustep@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-gnustep

[bug #52494] Does not handle exceptions when loading translations

[Yavor Doganov]

URL:
  

 Summary: Does not handle exceptions when loading translations
 Project: GNUstep
Submitted by: yavor
Submitted on: Fri 24 Nov 2017 04:36:29 PM EET
Category: Gorm
Severity: 3 - Normal
  Item Group: Bug
  Status: None
 Privacy: Public
 Assigned to: None
 Open/Closed: Open
 Discussion Lock: Any

___

Details:

I tried to translate a .gorm file using Gorm's Document->Translate feature. 
Exported the strings fine but couldn't load the translated .strings file.  I
selected the file in the open panel, pressed OK, the panel disappeared but
nothing happened.  It took me some time to figure out what's going on...

The exported strings are unquoted when there is only one word and I simply
forgot to quote one translated string.  So the exception that is rasied in
GSPropertyListFromStringsFormat is not caught by Gorm.  IMHO this is not
proper behavior: Gorm should report the error, otherwise the user is left
wondering what actually happened.  Also, you should not assume that
translators are aware that strings must be quoted, so I think it is a good
idea to provide them with some hint (using the established practice of
conveying information with a comment starting with "TRANSLATORS:").

Attached is a patch that fixes this issue.  I see that the ChangeLog has not
been updated recently, but here is a suggested entry anyway:

2017-11-24  Yavor Doganov  

* GormCore/GormDocument.m (translate:): Add exception handling
when loading translations.
(exportStrings:): Prepend notice for translators.



___

File Attachments:


---
Date: Fri 24 Nov 2017 04:36:29 PM EET  Name:
0001-Add-exception-handling-when-loading-translations.patch  Size: 2KiB   By:
yavor



___

Reply to this item at:

  

___
  Message sent via/by Savannah
  http://savannah.gnu.org/


___
Bug-gnustep mailing list
Bug-gnustep@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-gnustep