Edgewall Software

Changes between Initial Version and Version 1 of Ticket #11189, comment 24


Ignore:
Timestamp:
Sep 18, 2013, 6:34:40 AM (11 years ago)
Author:
Ryan J Ollos

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #11189, comment 24

    initial v1  
    88However, since `ConfigurationError` is defined in the `config` module, your narrower definition seems more valid: // Exception raised when a value in the configuration file is not valid.// Maybe I should be using a `ParseError` for 198 and 201? `ParseError` should probably be located in a different module such as `trac.util` in that case. Maybe we should have an `InstallationError` exception for issues such as 183 and 189?
    99
    10 Maybe it doesn't matter at all and I'm worrying about something irrelevant? At this point, the major differentiator I can see is having a more self-explanatory title when the exception is presented. As comment:4:ticket:11272 shows a //Configuration Error//, the `InstallationError` would display //Installation Error//.
     10Maybe it doesn't matter at all and I'm worrying about something irrelevant? At this point, the major differentiator I can see is having a more self-explanatory title when the exception is presented. As comment:4:ticket:11272 shows a //Configuration Error//, the `InstallationError` exception could display //Installation Error//.