Changes between Initial Version and Version 1 of Ticket #11189, comment 24
- Timestamp:
- Sep 18, 2013, 6:34:40 AM (11 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #11189, comment 24
initial v1 8 8 However, 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? 9 9 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//.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` exception could display //Installation Error//.