Changeset 1767 for branches/eraser6/CodeReview/Eraser.Manager/Task.cs

Timestamp:

2/8/2010 7:09:55 AM (2 years ago)

Author:

lowjoel

Message:

Do not localise exceptions which aren't meant to be seen by the user. User-visible exceptions will be dealt with in a separate ticket. Do not throw exceptions to indicate handleable errors (notably, when erasing unused space without administrator permissions) Addresses #275: Code Review

File:

• branches/eraser6/CodeReview/Eraser.Manager/Task.cs (modified) (1 diff)

branches/eraser6/CodeReview/Eraser.Manager/Task.cs

@@ -183 +183 @@
             {
                 if (value.Owner != null)
-                    throw new ArgumentException(S._("The schedule provided can only " +
-                        "belong to one task at a time"));
+                    throw new ArgumentException("The schedule provided can only " +
+                        "belong to one task at a time");
 
                 if (schedule is RecurringSchedule)