#5080 Check **kwargs on ming find() NEEDS RELEASE

forge-nov-02
closed
stability (32)
General
1
2012-11-02
2012-10-06
Dave Brondsema
No

Ming's get() supports **kwargs but find() does not. Ensure that find() requires at least one positional arg for the criteria, rather than returning everything in the collection. Ming does a few checks like that already in other places.

Prevent https://engr.geek.net/git/?p=forge-classic.git;a=commitdiff;h=373110db792b11e0e5f82ded08c0c36e12b82a39 from recurring

Related

Tickets: #5248

Discussion

  • Dave Brondsema
    Dave Brondsema
    2012-10-30

    merciless:db/5080 and allura:db/5080 After review, we need to cut a ming release and update it in req.txt

     
  • Cory Johns
    Cory Johns
    2012-10-31

    • summary: Check kwargs on ming find() --> Check kwargs on ming find() NEEDS RELEASE
    • status: code-review --> validation
     


Anonymous


Cancel   Add attachments