bugphpGroupWare - Bugs: bug #19447, On SOAP 'fault' method result...

 
 

You are not allowed to post comments on this tracker with your current authentification level.

bug #19447: On SOAP 'fault' method result errors are in the page

Submitted by:  Olivier Berger <olberger>
Submitted on:  Thu 29 Mar 2007 07:54:14 PM UTC  
 
Item Group: 0.9.16.011Category: soap
Severity: 3 - NormalPriority: 4
Status: Need InfoPrivacy: Public
Assigned to: NoneOpen/Closed: Open
Component Version: SVNOperating System: None
Reproducibility: IntermittentPlanned Release: None
Fixed Release: 

Wed 05 Sep 2007 03:54:27 PM UTC, comment #1:

Hi Olivier,

If you can give me more info, it could be great :p

Regards,

Caeies.

Caeies <Caeies>
Project Administrator
Thu 29 Mar 2007 07:54:14 PM UTC, original submission:

In the code recently committed into the CVS on the 0_9_16 branch, there subsist "print" code in the call() method, which will report in the page any 'fault' return code from the SOAP method invoked on the server.

I guess this is not a big worry since not a lot must use SOAP, but nevertheless, I think it would be much better to hold the error in some "errno" kind of variable in the soap client, have call() return false, and let the caller check the error later on instead of polluting the display..

My 2 cents,

Best regards,

Olivier Berger <olberger>

 

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by Caeies (Posted a comment)
  • -unavailable- added by olberger (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follows 1 latest change.

    Date Changed By Updated Field Previous Value => Replaced By
    Wed 05 Sep 2007 03:54:27 PM UTCCaeiesStatusNone=>Need Info

    Back to the top


    Powered by Savane 3.1-cleanup