[PEAR-BUG] Bug #20490 [Dup]: Redundant LISTSCRIPTS call

Edit report at https://pear.php.net/bugs/bug.php?id=20490&edit=1

 ID:               20490
 Updated by:       [email protected]
 Reported By:      alec at alec dot pl
 Summary:          Redundant LISTSCRIPTS call
 Status:           Duplicate
 Type:             Bug
 Package:          Net_Sieve
 Package Version:  1.3.3
 PHP Version:      Irrelevant
 Roadmap Versions: 
 New Comment:

It's redundant, but I figured it's not too expensive. Don't you agree?


Previous Comments:
------------------------------------------------------------------------

[2015-01-20 18:25:45] alec

I'd prefer to have it fixed not removed. We end up with redundant
LISTSCRIPTS call.

------------------------------------------------------------------------

[2015-01-20 18:15:40] yunosh

-Status: Open
+Status: Duplicate
Duplicate of bug #20472 (which unfortunately got lost in the ticket
system).

------------------------------------------------------------------------

[2015-01-20 18:06:53] alec

Description:
------------
I think it is a common scenario to call listScripts() and then
getActive(). There's already an optimization with $this->_active check.
However, this does not work if there's no active script.

BTW, _cmdSetActive() should set $this->_active not
$this->_activeScript.
BTW2, this internal variable is undocumented.

------------------------------------------------------------------------


-- 
Edit this bug report at https://pear.php.net/bugs/bug.php?id=20490&edit=1