chuushi / CoreProtect-Lookup-Web-Interface

A flexible lookup web interface for CoreProtect 2
MIT License
50 stars 14 forks source link

World filter doesn't work with Coords filter #19

Closed snarfattack closed 4 years ago

snarfattack commented 6 years ago
       if (isset($coord)) $filter['coord'] .= " AND ";
        else               $filter['coord'] = "";
                           $filter['coord'] .= "wid=".$Cc->getId($q['wid'],"world");

If evaluation will always resolve to false since $coord does not exist, the code should be:

        if (isset($filter['coord'])) $filter['coord'] .= " AND ";
        else $filter['coord'] = "";
        $filter['coord'] .= "wid=".$Cc->getId($q['wid'],"world");
chuushi commented 4 years ago

I have since rewritten the code to handle them separately. The world and coords filters should be independent of each other in the latest version!

chuushi commented 4 years ago

Coordinates and worlds now gets looked up independently of each other. Thank you for the issue! (Commit at close: e5b36f36d79bfcea6891ca0b2f95266ead01b0fd)