I lose the class but i found an other one with that problem :D Into Pillar, inside the Pillar-Model package you have the class PRFileInclusion with only 3 methods: PRFileInclusion>>#aboutClass ^ PRInputFileAnnotation PRTransformerWithoutUselessParagraph>>#aboutClass ^ self shouldBeImplemented ____________________________________ PRFileInclusion>>#specificVisitor: anInputFileAnnotation ^ self visitInputFileAnnotation: anInputFileAnnotation PRTransformerWithoutUselessParagraph>>#specificVisitor: something ^ self shouldBeImplemented _____________________________________ PRFileInclusion>>#visitInputFileAnnotation: anInputFileAnnotation "I load the file and if the file exist I replace the node of the annotation by the content of the file." | file | file := anInputFileAnnotation fileWithConfiguration: self configuration. file exists ifTrue: [ | parsedFile | parsedFile := anInputFileAnnotation parseFile: file withConfiguration: self configuration. self replace: parsedFile children] ifFalse: [ anInputFileAnnotation errorFileNotFound: file ] PRVisitor>>#visitInputFileAnnotation: anInputFileAnnotation self visitAnnotation: anInputFileAnnotation http://puu.sh/hnODJ/6595a1cdb6.jpg On 23 April 2015 at 08:40, Yuriy Tymchuk <yuriy.tymchuk@me.com> wrote:
Hi,
thank you for reporting. Can you tell me on which project are you running it, or file out the hierarchy so I can reproduce it?
Uko
On 22 Apr 2015, at 18:06, Cyril Ferlicot <cyril.ferlicot@gmail.com> wrote:
Hi ! Your quality assistant say to me "repeated method in the superclass" (If a class is overriding a method, it should use a different code. Is meaningless to have a method in a class and in its superclass) but i've 3 methods into my class and all overrides a method with "^ shouldBeImplemented" in the superclass. I think something is wrong with this warning :)
On 22 April 2015 at 16:45, Yuriy Tymchuk <yuriy.tymchuk@me.com> wrote:
Good point, this brings up two problems.
1) The rule can improve detection by checking the type is the message is sent to self 2) Automatic fixing should display the changes and letting you decide whether you want to apply them.
1st will take some time. 2nd will be soon in QA :)
Uko
On 22 Apr 2015, at 16:33, Johan Fabry <jfabry@dcc.uchile.cl> wrote:
Some weird behavior in quality assistant. Have a look at this method:
duplicateVar: aVarName
self show: 'Var: ', aVarName, ' redefinition ignored'.
This raises the issue: âUse cascaded nextPutAll:âs instead of #, in nextPutAll:â It also pops up the automatic rewrite icon, so I clicked on it to see how it would transform the code. Result:
duplicateVar: aVarName self show: 'Var: ' , aVarName; show: ' redefinition ignored'
Needless to say, thatâs not a behavior preserving transformation.
On Apr 22, 2015, at 03:58, Yuriy Tymchuk <yuriy.tymchuk@me.com> wrote:
Please give me a feedback about your experience, as I want to make it even more useful. Either write me an email, or open an entry at: https://github.com/Uko/QualityAssistant/issues
---> Save our in-boxes! http://emailcharter.org <---
Johan Fabry - http://pleiad.cl/~jfabry PLEIAD lab - Computer Science Department (DCC) - University of Chile
-- Cheers Cyril Ferlicot
-- Cheers Cyril Ferlicot