All users were logged out of Bugzilla on October 13th, 2018

contrib/bz_webservice_demo.pl fails with --product_name (Product.get_products doesn't accept product names)

VERIFIED INVALID

Status

()

VERIFIED INVALID
11 years ago
11 years ago

People

(Reporter: Frank, Unassigned)

Tracking

Details

(Reporter)

Description

11 years ago
User-Agent:       Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10_5_2; de-de) AppleWebKit/525.13 (KHTML, like Gecko) Version/3.1 Safari/525.13
Build Identifier: Safari Version 3.1 (5525.13)

contrib/bz_webservice_demo.pl fail with 
Can't use string ("versionResponse") as a HASH ref while "strict refs"

Reproducible: Always

Steps to Reproduce:
1.install Bugzilla version 3.1.3
2. run the contrib/bz_webservice_demo.pl with the location of the installed Bugzilla as uri and --login  --password  with valid values and pass --product_name TestProduct
Actual Results:  
Subroutine LWP::UserAgent::redirect_ok redefined at /Library/Perl/5.8.8/SOAP/Transport/HTTP.pm line 41.
Subroutine LWP::Protocol::collect redefined at /Library/Perl/5.8.8/SOAP/Transport/HTTP.pm line 59.
Can't use string ("versionResponse") as a HASH ref while "strict refs" in use at /usr/local/bugzilla-dev/contrib/bz_webservice_demo.pl line 184.



Expected Results:  
Infos of the TestProduct

Comment 1

11 years ago
The bug reported in comment 0 is a local problem. You seem to have SOAP::Lite incorrectly installed. So this part is WFM (or even INVALID).

Now, even with SOAP::Lite correctly installed, bz_webservice_demo.pl still fails when called with --product_name defined as it tries to call Product.get_product() with the product name as argument, but this API has been rename to Product.get_products() (plural!) and now only accepts a list of product IDs.

I think the current API is wrong! You have no way to guess the ID of a product, and passing a product name to the API makes *much more* sense than passing a list of IDs. So Product.get_products() should still be able to accept a (list of) product name(s). I consider this as a regression as it was working fine (i.e. in a "logical" manner) till 2.23.2. We should fix this API before 3.2 RC1 IMO. We should also fix it on the 3.0 branch as it won't break its current behavior.
Status: UNCONFIRMED → NEW
Depends on: 357482
Ever confirmed: true
Keywords: regression
OS: Mac OS X → All
Priority: -- → P2
Hardware: Macintosh → All
Summary: contrib/bz_webservice_demo.pl fail → contrib/bz_webservice_demo.pl fails with --product_name (Product.get_products doesn't accept product names)
Target Milestone: --- → Bugzilla 3.0
Version: unspecified → 3.1.3

Comment 2

11 years ago
(In reply to comment #1)
> (i.e. in a "logical" manner) till 2.23.2.

2.23.3 even.
(Reporter)

Comment 3

11 years ago
(In reply to comment #1)
> The bug reported in comment 0 is a local problem. You seem to have SOAP::Lite
> incorrectly installed. So this part is WFM (or even INVALID).
> 
Sorry, I know that ther was no error during installation of SOAP::Lite.

But if you think that I should reinstall, please tell me what I must do.

Thanks
(In reply to comment #1)
> I think the current API is wrong! You have no way to guess the ID of a product,
> and passing a product name to the API makes *much more* sense than passing a
> list of IDs. So Product.get_products() should still be able to accept a (list

I agree. We should not drop support of passing IDs, though. The web service is primarily focused to be accessed in some automated way. IDs have the advantage that the referred object cannot "wiggle away" by being renamed: it's still the *same product* after a rename. This is helpful for all programs which keep persistent object information with themselves.
OS: All → Mac OS X
Priority: P2 → --
Hardware: All → Macintosh
Target Milestone: Bugzilla 3.0 → ---
Version: 3.1.3 → unspecified

Comment 5

11 years ago
I think your problem is a bug in SOAP::Lite, which I have reported to the maintainer:

  http://rt.cpan.org/Public/Bug/Display.html?id=34516
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → INVALID

Comment 6

11 years ago
Forget comment 0, read comment 1.
Status: RESOLVED → REOPENED
Resolution: INVALID → ---

Comment 7

11 years ago
Restoring changes.
Status: REOPENED → NEW
OS: Mac OS X → All
Hardware: Macintosh → All
Target Milestone: --- → Bugzilla 3.0
Version: unspecified → 3.1.3

Comment 8

11 years ago
File another bug for comment 1--this user's problem is a bug in SOAP::Lite.
Status: NEW → RESOLVED
Last Resolved: 11 years ago11 years ago
Keywords: regression
Resolution: --- → INVALID
Target Milestone: Bugzilla 3.0 → ---

Comment 9

11 years ago
(In reply to comment #8)
> File another bug for comment 1--this user's problem is a bug in SOAP::Lite.
> 

I filed bug 428406.
Status: RESOLVED → VERIFIED
No longer depends on: 357482
You need to log in before you can comment on or make changes to this bug.