Bug #358
closed
- Status changed from New to In Code Review
- Assignee changed from Eric Hou to Amber Herold
There is a problem on checking out the php error message. First pregmatch function should be preg_match. And there is not CATCH_PHPERRMSG constant get defined.
I just comment out this three lines now to make it work.
- Status changed from In Code Review to In Test
- Assignee changed from Amber Herold to Neil Voss
- Target version set to Appion/Leginon 2.0.0
r13557
I really don't like the idea of reducing the level of error checking to fix a problem. Neil, do you see any problem with this and can you test it?
- Assignee changed from Neil Voss to Eric Hou
I think we had no clue what the error check was for, but it broke things. Or maybe it has to php_errormsg NOT phperrormsg. Check with Eric.
- Assignee changed from Eric Hou to Amber Herold
- % Done changed from 0 to 100
The problem was there were syntax error in the code. The person who put this extra error checking wasn't actually do it right, so I don't know what error he was try to catch.
We did not write this code, it was download from other open source project. People just add extra error checking for their own good. So don't waste time for this unless you have nothing to do or you have some better ideas to solve it.
Thanks.
Eric
- Status changed from In Test to Closed
Also available in: Atom
PDF