On Sun, Apr 06, 2014 at 06:21:05PM -0700, m4tu5 wrote:
> > @@ -92,13 +92,6 @@ class RecibosControllerTest < ActionController::TestCase
> >      assert_response :success
> >    end
> >  
> > -  test "no muestra a través de otra obra" do
> 
> es de recibos el test, pero si, se puede hacer lo que decis, si cargas el recibo a mano con una URL equivocada. 
> 
> mas arriba tuve que modificar el controlador de recibos, porque como no es
> facil asociar a los internos con obras, mantener el filtro por obras estaba
> complejo ... y me parecía innecesario, ya que esta buscando el recibo por id,
> y va a encontrar solo uno y siempre el mismo.

estos tests están porque vos te quejaste (con razón) de que se podía llegar a
facturas de obras diferentes a la de la url poniendo el id. Este es el mismo
caso.. por qué está bien acá pero no en facturas? seamos coherentes.. es
cuestión de arreglar lo que esté mal. Si entiendo bien, es culpa de que los
recibos internos no llegan a una obra, si no es a través de factura? Hacemos
una especialización de recibo ahí?

--