Registro se guarda por duplicado

[Maru77]

Hola a todos!

Tengo un inconveniente, al agregar un registro lo guarda por duplicado de la siguiente manera

actualPorcentaje 10 | marcaid 0
actualPorcentaje 0 | marcaid 4

Este formulario me permite actualizar los precios a través de un porcentaje, previo seleccionar una marca

Código HTML:

<form id="formProductos" action="<?php echo base_url

() ?>index.php/actualizacion/adicionar" method="post">

<label for="">Porcentaje a actualizar</label>
            <input type="text" placeholder="Escriba el % de actualización" id="actualPorcentaje" name="actualPorcentaje"  />
                               
</form> 

Código PHP:

  <?php 
 echo '<a id="enviar" href="'.base_url 
 
().'index.php/actualizacion/adicionar/'.$marcaid.'"> Actualizar Precios</a>';  
?>   

Código HTML:

<script type="text/javascript">

$(document).ready(function(){
	
   $("#enviar").click(function(event){		
		
		$.ajax({
		type: "POST",
		data: $('#formProductos').serialize(),
		
		url: "<?php echo base_url();?>index.php/actualizacion/adicionar",
		
		dataType: 'json',
		
		success: function(data)
						   {
							

if(data.result == true){							

	$('.results').html(data);							

}							

else{							

	alert('Ocurrió un error al agregar un producto.');							

}
							
</script> 

función para insertar registros in model.php

Código PHP:

   function add($table,$data){ 
        $this->db->insert($table, $data);          
        if ($this->db->affected_rows() == '1') 
        { 
            return TRUE; 
        } 
         
        return FALSE;        
    }  
   

función para adicionar registros en controller.php

Código PHP:

  function adicionar(){ 
 
    $this->load->library('pagination'); 
 
         
        $config['base_url'] = base_url().'index.php/actualizacion/gerenciar/'; 
 
        $config['total_rows'] = $this->actualizacion_model->count('actualprecios'); 
 
        $config['per_page'] = 10; 
 
        $config['next_link'] = 'Próxima'; 
 
        $config['prev_link'] = 'Anterior'; 
 
        $config['full_tag_open'] = '<div class="pagination alternate"><ul>'; 
 
        $config['full_tag_close'] = '</ul></div>'; 
 
        $config['num_tag_open'] = '<li>'; 
 
        $config['num_tag_close'] = '</li>'; 
 
        $config['cur_tag_open'] = '<li><a style="color: #2D335B"><b>'; 
 
        $config['cur_tag_close'] = '</b></a></li>'; 
 
        $config['prev_tag_open'] = '<li>'; 
 
        $config['prev_tag_close'] = '</li>'; 
 
        $config['next_tag_open'] = '<li>'; 
 
        $config['next_tag_close'] = '</li>'; 
 
        $config['first_link'] = 'Primeira'; 
 
        $config['last_link'] = 'Última'; 
 
        $config['first_tag_open'] = '<li>'; 
 
        $config['first_tag_close'] = '</li>'; 
 
        $config['last_tag_open'] = '<li>'; 
 
        $config['last_tag_close'] = '</li>'; 
 
             
 
        $this->pagination->initialize($config);      
 
        $this->data['results'] = $this->actualizacion_model->get('actualprecios','idActualizacion, actualPorcentaje,actualFecha','',$config['per_page'],$this->uri->segment(3)); 
         
         
        $this->load->library('form_validation'); 
 
        $this->data['custom_error'] = '';  
       
             
        $marcaid = $this->uri->segment(3); 
             
            $this->data['productos'] = $this->actualizacion_model->getProductosById($marcaid); 
             
            $productos = $this->data['productos']; 
                                                        
            $dataInicial = date("Y").'-'.date("m").'-'.date("d"); 
             
            $usuario = $this->session->userdata('logado'); 
             
            $actualPorcentaje = $this->input->post('actualPorcentaje'); 
                         
             
            $data = array(                 
                 
                'actualPorcentaje' => $actualPorcentaje , 
                 
                'actualFecha' => $dataInicial, 
                 
                'marcaid' => $marcaid, 
                 
                'usuarioid' => $usuario,  
                ); 
 
 
 
           if ( is_numeric($id = $this->actualizacion_model->add('actualprecios', $data, true)) ) { 
                 
                 
                foreach ($productos as $p) { 
                    $idProdutos = $p->idProdutos;         
                     
                    $precoVenda = $p->precoVenda; 
                     
                    $precioAnterior = $p->precoVenda; 
                     
                    $precoVenda = $precoVenda *(1 + $actualPorcentaje / 100); 
                     
                    $sql = "UPDATE produtos set precioAnterior = ?, precoVenda = ? WHERE idProdutos = ?"; 
             
                    $this->db->query($sql, array($precioAnterior, $precoVenda, $idProdutos)); 
                     
                } 
                                         
                 
                $this->session->set_flashdata('success','El registro de Actualización de Precios se agregó correctamente!!!'); 
                 
                redirect(base_url().'index.php/actualizacion/gerenciar/');      
                 
            } else { 
                 
                $this->data['custom_error'] = '<div class="form_error"><p>An Error Occured.</p></div>'; 
 
            }         
 
 
        $this->data['view'] = 'actualizacionp/actualizacion'; 
 
        $this->load->view('tema/topo', $this->data); 
 
    } 
  
                          } 
        }); 
   }); 
 
});  
   

[xlos_mas_pro]

Yo te recomiendo cambiar el click por un submit del formulario y le quitas la acción con event.preventDefault(); luego bloqueas el boton de submit al enviarlo con un disable desde el javascript mientras se envia tu consulta ajax y luego lo vuelves a habilitar al recibir la confirmación

Además fíjate a escribir las consultas a la base dentro de un foreach