current community

your communities

more stack exchange communities

Stack Exchange
tour help
careers 2.0
Take the 2-minute tour ×
Stack Overflow is a question and answer site for professional and enthusiast programmers. It's 100% free, no registration required.
up vote 0 down vote favorite
1

At this moment i access my posts like this 

http://localhost/post/174565/

and everything works fine, but when i am trying to access them like this 

http://localhost/post/1745s65/

i am getting error

Error Number: 1054
Unknown column '1745s65' in 'where clause'
SELECT * FROM posts WHERE id = 1745s65;
Filename: D:\Localhost\Apache\htdocs\code\system\database\DB_driver.php
Line Number: 330

i understand why it is there, but how to handle it? For example my users don't need to see SQL query, and i want to show them 404 page instead of this block.

share|improve this question
    
post your code that queries the db based on the url. –  Sport Billy Mar 30 '13 at 23:19
    
for what you need it? post is not about debuging sql query, but handling errors :) –  Nerfair Mar 30 '13 at 23:42
1  
to see the way that you parse the query. Normally, if you used active record correctly you should not see the above error. –  Sport Billy Mar 30 '13 at 23:43
    
Ahh, ok, here it is pastebin.com/aPt7VYuJ –  Nerfair Mar 30 '13 at 23:45

3 Answers 3

up vote 1 down vote accepted

Try this code:

        public function getPostData($iPostId) {
            $sqlQuery = $this->db->get_where('posts', array('id' => $iPostId));
            $aResult = $sqlQuery->result();
            if (empty($aResult)) {
                    show_404();
            }
            else {
                    return $aResult[0];
            }
    }
share|improve this answer
    
Magic method! Works just perfectly. –  Nerfair Mar 31 '13 at 11:56
up vote 0 down vote

You need to validate user input to prevent this issues. Anyhow, Codeigniter has two preset environments: development and prodution. Check your root index.php for more information.

share|improve this answer
    
I thought about checking is_numeric and if SQL result !empty, thought there are some built-in functions :) Production environment changes nothing in this situation –  Nerfair Mar 30 '13 at 23:40
up vote 1 down vote

use execptions:

try{
       //here what you need to do with script
         } catch (Exception $e) {
            var_dump($e->getMessage());
            show_404();
              }

i think you can try also to put :

$database['db_debug'] = FALSE; in config/database.php if you want to remove database errors from being shown

share|improve this answer

Your Answer

 
discard

By posting your answer, you agree to the privacy policy and terms of service.

Not the answer you're looking for? Browse other questions tagged or ask your own question.