Bug #6071 for WWW-Mechanize: return value of ->set

Mon Apr 19 14:23:32 2004 mjd [...] porpoise.net.isc.upenn.edu - Ticket created

Date:	Mon, 19 Apr 2004 14:23:36 -0400 (EDT)
From:	mjd [...] porpoise.net.isc.upenn.edu (Mark Dominus)
To:	undisclosed-recipients: ;

Download (untitled) / with headers
text 339b

Message body not shown because it is not plain text.

Thu May 27 16:18:54 2004 andy [...] petdance.com - Subject changed from '[no subject]' to 'return value of ->set_visible is not documented'

Tue Oct 19 22:32:25 2004 MARKSTOS [...] cpan.org - Taken

Tue Oct 19 22:57:42 2004 MARKSTOS [...] cpan.org - Correspondence added

RT-Send-CC:

petdance

[mjd@porpoise.net.isc.upenn.edu - Mon Apr 19 14:23:32 2004]: Show quoted text

> Re version1.02: > > The return value of ->set_visible is not documented. > If ->set_visible hass no meaningful return value, I suggest > that you make it return success when it was successfully > able to set the form fields, and failure otherwise. > > For example, ->set_visible("a", "b") might fail on a form with > only one field. > ~b mjd@plover.com

How about this: # return the set and unset that were passed in. if (my ($set,$unset) = $mech->set_visible('dogs','cats') and !$unset # all inputs were set! } # else { # examine unset inputs to debug. } ##### That way, if setting any value doesn't happen, you have a way to track down exactly which one it was. Mark

Tue Oct 19 22:57:43 2004 MARKSTOS [...] cpan.org - Status changed from 'new' to 'stalled'

Sat Jul 15 21:38:50 2006 MARKSTOS [...] cpan.org - Correspondence added

A couple of years later, the return value of set_visible() is now documented, according to what the code has been doing. I recommend resolving this ticket. Mark

Sat Jul 15 21:38:51 2006 The RT System itself - Status changed from 'stalled' to 'open'

Sat Jul 15 22:10:50 2006 PETDANCE [...] cpan.org - Status changed from 'open' to 'resolved'

Bug #6071 for WWW-Mechanize: return value of ->set_visible is not documented

Preferred bug tracker