refactor the test about $limit ===0
greggles - December 18, 2008 - 18:49
| Project: | User Quota |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | task |
| Priority: | critical |
| Assigned: | greggles |
| Status: | closed |
Jump to:
Description
1. The first test should be for "unlimited"
2. The second can be about it being 0 or negative
3. Then we know that the rest is positive numbers
Also, use a defined constant instead of the "unlimited" string
#1
And that.
#2
Though I forgot to include the nid, it was fixed in http://drupal.org/cvs?commit=159933
#3
Automatically closed -- issue fixed for two weeks with no activity.