http://bugzilla.wpkg.org/show_bug.cgi?id=175

           Summary: Test return value of removeSettingsNode function
           Product: WPKG
           Version: other
          Platform: PC
        OS/Version: Windows XP
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: wpkg.js
        AssignedTo: man...@wpkg.org
        ReportedBy: siwalt...@hotmail.com
         QAContact: wpkg-users@lists.wpkg.org


removeSettingsNode is used at 4 places in V1.1.2 (Line 614 , Line 4015,Line
4093 and Line 4119) but no test is made of its return falue (true/false) and
usually WPKG just prints a package removal success message.

I think it would be better to test its return value and suppress the successful
message and output a debug message if it returns false.

regards
Simon

-- 
Configure bugmail: http://bugzilla.wpkg.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
-------------------------------------------------------------------------
wpkg-users mailing list archives >> http://lists.wpkg.org/pipermail/wpkg-users/
_______________________________________________
wpkg-users mailing list
wpkg-users@lists.wpkg.org
http://lists.wpkg.org/mailman/listinfo/wpkg-users

Reply via email to