On 9 March 2012 01:22, Guillermo Polito <[email protected]> wrote: > ClassBuilder>>validateClassName: aString > "Validate the new class name" > | allowed | > aString isSymbol > ifFalse: [ ^ false ]. > allowed := ($0 to: $9), {$_}, ($A to: $Z), ($a to: $z). > (aString detect: [:c | (allowed includes: c) not] ifNone: [ ]) > ifNotNil: [ :c | self error: 'Invalid character: ''', c printString, > ''''. > ^ false]. > aString first canBeGlobalVarInitial ifFalse:[ > self error: 'Class names must be capitalized'. > ^false]. > environ at: aString ifPresent:[:old| > (old isKindOf: Behavior) ifFalse:[ > self notify: aString asText allBold, > ' already exists!\Proceed will store over it.' > withCRs]]. > ^ true > > > Is it right to do that asText allBold there? :S > Shouldn't an exception be raised instead? > > Do I open a ticket? > Yes. We should kill dependency by removing asText asBold. Notifications should not accept rich-formatted text anyways, because not all ui managers can handle it (now we having command-line ui manager which prints message on console).
> Guille -- Best regards, Igor Stasenko.
