Hi Nazir, thanks again for your help!
You are right. There is a missed method in the class TestObjectFactory.java. Thus, I included it in a new generated patch (hope it will work at this time). (I also attached the new patch file in the Issue Tracker). https://mifos.dev.java.net/nonav/issues/showattachment.cgi/431/patch_Issue_1447.patch Thanks Selim De : [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] De la part de Nazir LAJDEL Envoyé : lundi 4 février 2008 18:17 À : Developer Cc : [EMAIL PROTECTED] Objet : RE: [Mifos-developer] Patchs for issue #1446 and issue #1447 Hi again Selim, Concerning the second patch; when applying it and run ApplicationTestSuite, I got the following error; cannot find symbol [javac] symbol : method createNotAllowedProductForAProductOffering(org.mifos.application.productdefinition.business.PrdOfferingBO,org.mifos.application.productdefinition.business.PrdOfferingBO) It seems like method createNotAllowedProductForAProductOffering is not included in the patch. I wish you good luck with your next patch.. - Nazir De : [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] De la part de Nazir LAJDEL Envoyé : lundi 4 février 2008 17:08 À : Developer Cc : [EMAIL PROTECTED] Objet : Re: [Mifos-developer] Patchs for issue #1446 and issue #1447 Thank you Selim for your patches. I started reviewing the first one (#1446). Overall this patch looks good, However there's a feedback; There is a need to write tests for method getPrdOfferingMix() in class PrdOfferingPersistenceTest and getPrdOfferingMix in class ProductMixBusinessServiceTest, to ensure their coverage. Looking forward to an updated patch. Thanks, - Nazir De : [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] De la part de Selim Lachiheb Envoyé : jeudi 31 janvier 2008 15:53 À : [email protected] Objet : [Mifos-developer] Patchs for issue #1446 and issue #1447 Hi Nazir , Please find the 2 patchs for review related to : issue #1446 (https://mifos.dev.java.net/issues/show_bug.cgi?id=1446 <https://mifos.dev.java.net/issues/show_bug.cgi?id=1446> ) issue #1447 (https://mifos.dev.java.net/issues/show_bug.cgi?id=1447 <https://mifos.dev.java.net/issues/show_bug.cgi?id=1447> ) Please let me know if more information is required. Suggestions are welcome. With Regards, Selim _____________________________________________________________________ Selim LACHIHEB Software Developer OXIA OXIA is ISO 9001 V2000 certified company - AFAQ QUAL/2005/25140 11, Rue Claude Bernard 1002 Tunis Belvédère Tunisia Tel.: +216 71 28 27 00 Fax: +216 71 28 27 05 [EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]> www.oxiasoft.com <BLOCKED::http://www.oxiasoft.com/> --~--~---------~--~----~------------~-------~--~----~ Vous avez reçu ce message, car vous êtes abonné au groupe Groupe "OXIA_ENDA_MGMT" de Google Groupes. Pour transmettre des messages à ce groupe, envoyez un e-mail à l'adresse [EMAIL PROTECTED] Pour résilier votre abonnement à ce groupe, envoyez un e-mail à l'adresse [EMAIL PROTECTED] Pour afficher d'autres options, visitez ce groupe à l'adresse http://groups.google.com/group/oxia_enda_mgmt?hl=fr -~----------~----~----~----~------~----~------~--~---
patch_Issue_1447.patch
Description: patch_Issue_1447.patch
------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
