{"action":"created","issue":{"url":"https://api.github.com/repos/mradosta/pragtico/issues/13","repository_url":"https://api.github.com/repos/mradosta/pragtico","labels_url":"https://api.github.com/repos/mradosta/pragtico/issues/13/labels{/name}","comments_url":"https://api.github.com/repos/mradosta/pragtico/issues/13/comments","events_url":"https://api.github.com/repos/mradosta/pragtico/issues/13/events","html_url":"https://github.com/mradosta/pragtico/issues/13","id":163732719,"number":13,"title":"Tildes de tipo de liquidación en conceptos, posiblemente no funcionan correctamente","user":{"login":"JavierAder","id":20286980,"avatar_url":"https://avatars.githubusercontent.com/u/20286980?v=3","gravatar_id":"","url":"https://api.github.com/users/JavierAder","html_url":"https://github.com/JavierAder","followers_url":"https://api.github.com/users/JavierAder/followers","following_url":"https://api.github.com/users/JavierAder/following{/other_user}","gists_url":"https://api.github.com/users/JavierAder/gists{/gist_id}","starred_url":"https://api.github.com/users/JavierAder/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/JavierAder/subscriptions","organizations_url":"https://api.github.com/users/JavierAder/orgs","repos_url":"https://api.github.com/users/JavierAder/repos","events_url":"https://api.github.com/users/JavierAder/events{/privacy}","received_events_url":"https://api.github.com/users/JavierAder/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-07-04T19:05:20Z","updated_at":"2016-07-04T19:39:51Z","closed_at":null,"body":"Buenas, antes que nada me presento, ya que es mi primer posteo:\r\nSoy Javier Ader, y vengo hace unos meses trabajando con Pragtico, creo que corrigiendo algunos bugs e intentando encarar algunas modificaciones mucho más generales, entre ellas: hacerlo correr sobre nginx sobre Windows y así evitar Apache y sus instaladores poco mantenidos, e intentar migrar Pragtico a Cake 2.7 y así evitar versiones de php antiguas y problemas de performance... Realmente nunca tuve el tiempo para publicar estas cosas acá, así que pido disculpas y espero en cuanto pueda ir publicando lo que encontré y lo que vaya encontrando.\r\n\r\nOk, yendo al tema actual: siguiendo una issue publicado anteriormente y mirando el código, creo que hay algo que no funciona como \"se supone\" (entre comillas porque quizá sea ese justamente el funcionamiento correcto) con respecto a los tildes de de tipo de liquidación en la que se permite usar el mismo (normal, sac, final, etc).\r\nEn particular, cuando uno liquida sac o final (posiblemente también pase en otras tipos de liquidaciones), el foreach en la linea 145 (modelo liquidacion.php) https://github.com/mradosta/pragtico/blob/master/app/models/liquidacion.php#L445\r\n```\r\n            foreach ($this->Relacion->RelacionesConcepto->Concepto->findConceptos('Relacion',\r\n                    array(      'relacion'  => $relationship,\r\n                                'desde'     => $this->getVarValue('#fecha_desde_liquidacion'),\r\n                                'hasta'     => $this->getVarValue('#fecha_hasta_liquidacion'))) as $cCod => $concepto) {\r\n\t\t\t\tif (!$this->__isValid($concepto, array_keys($novedades['conceptos']))) {\r\n                    $this->__resolvConceptToZero($cCod);\r\n                } else {\r\n\t\t\t\t\t$this->setConcept(array($cCod => $concepto));\r\n\t\t\t\t}\r\n            }\r\n```\r\nsi entiendo bien hace lo siguiente: busca todos los conceptos a nivel de relación para la liquidación en curso, y luego los filtra si no corresponden para el tipo de liquidación que estamos haciendo. Ahora bien, el filtrado se hace a nivel de código mediantes __isValid(....) y esta función no hace lo que uno creería correcto.\r\nhttps://github.com/mradosta/pragtico/blob/master/app/models/liquidacion.php#L486\r\n```\r\n/**\r\n * Returns true when the concept is valid for the current receipt type, false in other cases.\r\n*/\r\n\tfunction __isValid($concept, $novelties) {\r\n\t\tif (((int)$concept['liquidacion_tipo'] & $this->__receiptTypeMapping[$this->__receiptType]) === $this->__receiptTypeMapping[$this->__receiptType]\r\n\t\t\t|| in_array($concept['codigo'], $novelties)\r\n\t\t\t|| $concept['imprimir'] == 'No'\r\n\t\t\t|| $concept['tipo'] == 'Deduccion') {\r\n\t\t\t//|| substr($concept['imprimir'], -9) === '[Forzado]') {\r\n\t\t\treturn true;\r\n\t\t} else {\r\n\t\t\treturn false;\r\n\t\t}\r\n\t}\r\n```\r\nparece ser que no mira SOLO en tipo de liquidación, hay excepciones; tambien considera valido (independientemente de los tildes de tipo de liquidación en el concepto, lo cual es chequeado en la primer condición del if)\r\nsi\r\n1) el concepto esta en la lista de novedades (supongo que eso esta perfecto)\r\no\r\n2) si es \"no imprimible\" (realmente no se si estaría bien o mal; creo que sí, porque esto debe estar pensado para \"conceptos auxiliares\" que en general solo se calculan para poder evaluar otros conceptos)\r\no\r\n3) si es de tipo Deducción\r\n\r\nEspecialmente para el caso 3 es mi duda: parece que los tildes para tipo de liquidación en los conceptos SOLO se aplican para conceptos Remunerativos o No Remunerativos, pero NO para de tipo Deducción. Otra forma de decirlo: las conceptos de tipo Deducción SIEMPRE son agregados a la liquidación independientemente de cual sea el tipo de la misma.\r\nSí esa es la idea, tal vez estaría bueno agregar este detalle en la documentación. \r\n\r\nFinalmente, aún si ese es el caso, vale aclarar (bah, creo) que uno siempre puede hacer que un concepto de tipo Deducción sea \"filtrado indirectamente\" mediante la formula, haciendo que se comporte como una deducción efectiva para las liquidaciones en las que deba aplicarse realmente y en los casos restantes que evalúe a cero (esto es exactamente lo mismo que hace __resolvConceptToZero).\r\nPor ej, digamos que se agrega concepto DeduccionNoEnSacOFinal que es una deduccion fija de 500 pesos que no se tiene que hacer en el sac y en e final; un podría poner la formula (basada en la formula de Obra Social que también es una deducción y por lo tanto se aplica siempre)\r\n`= if (or('#tipo_liquidacion' = 'sac', '#tipo_liquidacion' = 'final'), 0, 500)`\r\n\r\nTiene sentido hacer esto? O tal vez sería mejor modificar __inValid para que no haga la excpeción para las deducciones (esto es, que las filtre de igual manera que filtra a los restantes tipos)?\r\n\r\nGracias\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/mradosta/pragtico/issues/comments/230346149","html_url":"https://github.com/mradosta/pragtico/issues/13#issuecomment-230346149","issue_url":"https://api.github.com/repos/mradosta/pragtico/issues/13","id":230346149,"user":{"login":"mradosta","id":33546,"avatar_url":"https://avatars.githubusercontent.com/u/33546?v=3","gravatar_id":"","url":"https://api.github.com/users/mradosta","html_url":"https://github.com/mradosta","followers_url":"https://api.github.com/users/mradosta/followers","following_url":"https://api.github.com/users/mradosta/following{/other_user}","gists_url":"https://api.github.com/users/mradosta/gists{/gist_id}","starred_url":"https://api.github.com/users/mradosta/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mradosta/subscriptions","organizations_url":"https://api.github.com/users/mradosta/orgs","repos_url":"https://api.github.com/users/mradosta/repos","events_url":"https://api.github.com/users/mradosta/events{/privacy}","received_events_url":"https://api.github.com/users/mradosta/received_events","type":"User","site_admin":false},"created_at":"2016-07-04T19:39:51Z","updated_at":"2016-07-04T19:39:51Z","body":"Javier, gracias por tomarte el tiempo de reportar en detalle el problema. Migrar a Cake 2.7 seria un gran avance para Pragtico, Te invito que hagas un fork con lo poco o mucho de avance que tengas y vemos si lo podemos encarar de una buena vez al tema.\r\n\r\nCon respecto al reporte, a priori me atrevo a decir que es un BUG y debe modificarse el metodo __isValid, pero es un bug \"hecho a drede\", por lo que me hace dudar. Voy a hacer algunas pruebas y vuelvo con mas detalles."}}
{"actor":{"display_login":"mradosta"}}