would fail every time.
Added a missing DESTROY method to Group.pm. That way it won't puke when trying
to call an undefined method when the User object is cleaned up.
Added coverage tests for friends method in User.pm
a profile field existed in the db always passed because it used quickArray in
scalar context. Since quickArray always returns 1 element, this was always true
and it never returned undef like it was supposed to.
To fix this, I added a new method called quickScalar that returns the first column
from the row as a scalar. It has a basic test in SQL.t.
Then there's the new test in User.t for coverage that exposed this problem, and
I also removed a whole slew of duplicate tests.
Uncovered and fixed a bug where $session->user instead of $self was used in canUseAdminMode
Added an explicit test in Config.pm to make sure that array refs can be passed to ->set.
- Added a "run" link to the scheduler and the running workflows listings to
aid in debugging workflow errors.
- fix: profile fields not validated by WebGUI::User
User.pm: Collapse all methods into getUsers(1) inside isInGroup
Group.pm: Disable the cache whenever a scratchFilter or ipFilter is assigned.
Begin updating Group documentation for changes to dbQuery
requested group. LDAP work is TBD, but would look exactly
like any of the other get*Users methods in Group.pm. Then
the LDAP code would be removed from Users->isInGroup.
User.t: better clean-up at end
Scratch.t: no newlines in test output
Session/Scratch.pm: convert to use placeholders
Ad.pm: Add missing line of POD.
Group.pm: add method to get all users in a group by karma.
User.pm: isInGroup to use getKarmaUsers.
such as a dbQuery. This would happened coincidently before, but required
doing a User->isInGroup and only added members who were searched for rather
than all members.
Modify Group.t to test this.
Update POD in SQL.pm.
Multiple IPs are separated by commas, whitespace is ignored.
In groups, that affects the ipFilter field, User::isInGroup.
In settings, that affects Settings::debugIp and Session::ErrorHandler::canShowDebug.
Fixed a bug in WebGUI::Utility::isInSubnet where Net::Subnets->check needs a scalar.
Modified t/User.t to use addresses in CIDR format.