<html> <head> <meta content="text/html; charset=windows-1252" http-equiv="Content-Type"> </head> <body text="#000000" bgcolor="#FFFFFF"> <div class="moz-cite-prefix">On 03/08/2016 02:34 PM, Lenka Doudova wrote: </div> <blockquote cite="mid:56DED4F5.3000608@redhat.com" type="cite"> <meta content="text/html; charset=windows-1252" http-equiv="Content-Type"> <div class="moz-cite-prefix">On 03/08/2016 12:43 PM, Pavel Vomacka wrote: </div> <blockquote cite="mid:56DEBADB.5050503@redhat.com" type="cite"> <meta content="text/html; charset=windows-1252" http-equiv="Content-Type"> <div class="moz-cite-prefix">On 02/23/2016 03:20 PM, Lenka Doudova wrote: </div> <blockquote cite="mid:56CC6A9D.5060309@redhat.com" type="cite">Hi, attached is patch providing missing test coverage for ID views in webUI. Lenka <fieldset class="mimeAttachmentHeader"></fieldset> </blockquote> Hi, Thank you for your patch. I have small one comment: The delete_associtaion() function duplicates code, it would be good to create auxiliary function and move that code there. NACK. -- Pavel^3 Vomacka <fieldset class="mimeAttachmentHeader"></fieldset> </blockquote> Hi, thanks for review, fixed patch attached. Lenka <fieldset class="mimeAttachmentHeader"></fieldset> </blockquote> Thank you, ACK. -- Pavel^3 Vomacka </body> </html>