Add div around validation_errors

Question

Hi i want to edit my validation_errors function that it will always have a div around it. Im using bootstrap and there is a nice way to show errors with Dismissable alerts Look here

I have found the function validation_errors() in system/helpers/form_helper.php which looks like:

if ( ! function_exists('validation_errors'))
{
    function validation_errors($prefix = '', $suffix = '')
    {
        if (FALSE === ($OBJ =& _get_validation_object()))
        {
            return '';
        }  
        return $OBJ->error_string($prefix, $suffix);
    }
}

How can i edit it that it returns :

     <div class="alert alert-danger alert-dismissable">
        errors here
     </div>

I know i shouldn't do it in a helper class because its bad when i update my codeigniter version, but i really do not know how it can be done in a diffrent way.

AbstractChaos · Accepted Answer · 2014-04-25T09:58:58.107

2

Have you tried

if ( ! function_exists('validation_errors'))
{
    function validation_errors($prefix = '', $suffix = '')
    {
        if (FALSE === ($OBJ =& _get_validation_object()))
        {
            return '';
        }  
        return '<div class="alert alert-danger alert-dismissable">'.$OBJ->error_string($prefix, $suffix).'</div>'
    }
}

or

<?php 
if(!empty(validation_errors())) {
   echo '<div class="alert alert-danger alert-dismissable">'.validation_errors().'</div>';
}
?>

or if you really need to modify the helper

if ( ! function_exists('validation_errors'))
    {
        function validation_errors($prefix = '', $suffix = '')
        {
            if (FALSE === ($OBJ =& _get_validation_object()))
            {
                return '';
            }

            if($OBJ->error_string($prefix, $suffix) != '') {
              return '<div class="alert alert-danger alert-dismissable">'.$OBJ->error_string($prefix, $suffix).'</div>';
            }
            else {
              return '';
            }
        }
    }

edited Apr 25 '14 at 09:58

answered Apr 25 '14 at 09:17

AbstractChaos

4,211
1
19
28

Yes but it show the div, even when there are 0 errors. – Sven van den Boogaart Apr 25 '14 at 09:21
Thats what i want to avoid, i dont want to type the error div in every view. i would need to change every view if i changed something in the alert div. if i can do it by default i wont need to change every view. – Sven van den Boogaart Apr 25 '14 at 09:24
that gives me an error: Can't use method return value in write context in line 604 , line 604 is if(!empty($OBJ->error_string($prefix, $suffix))) { thanks for the help so far ;) @AbstractChaos – Sven van den Boogaart Apr 25 '14 at 09:32
Works if i change it to if($OBJ->error_string($prefix, $suffix) != '') { if you edit it in ur answer i will accept ;) – Sven van den Boogaart Apr 25 '14 at 09:34
@SvenB hadnt spotted that, its due to http://stackoverflow.com/questions/1075534/cant-use-method-return-value-in-write-context/4328049#4328049 – AbstractChaos Apr 25 '14 at 10:03

Add div around validation_errors

1 Answers1