Merge lp:~openerp-mexico-maintainer/openerp-mexico-localization/7.0-ir_attachment_path_file_added-dev-yzk into lp:openerp-mexico-localization/7.0

[Isaac López Zúñiga]

Se dejan las líneas de código que permiten generar el path del archivo temporal, se mantiene el uso de Superuser_id

[Nhomar - Vauxoo (nhomar)]

Isaac.

Puedes colocar aquí ¿Por qué? se hace éste cambio por favor.

Lo técnicamente disponible está claro.

Saludos.

review: Needs Information

[Isaac López Zúñiga (isaako34)]

> Isaac.
>
> Puedes colocar aquí ¿Por qué? se hace éste cambio por favor.

     El código que se esta proponiendo para merge ya existía en la revno 306, la cual crearba un path para escribir los archivos temporales y evitar problemas cuando el servidor de Openerp se levanta como demonio y hay problemas de permisos para acceder a la carpeta raiz. Sin embargo en ese mismo cambio se dejo de usar Superuser_id, generando problemas para imprimir el reporte PDF. En la revno 309 se revirtieron todos los cambios de la revno 306, lo cual no debió haber sido asi, solo se tenia que haber cambiado uid por Superuser_id.

     Por lo tanto, si revisamos el diff de la revno 306 con esta propuesta, la única diferencia que encontraremos es el cambio de uid por SUPERUSER_ID.

> Lo técnicamente disponible está claro.
>
> Saludos.

[Nhomar - Vauxoo (nhomar)]

Ok excelente.
Gracias Viejo!

El 9 de octubre de 2013 10:29, Isaac López Zúñiga - http://www.vauxoo.com <
<email address hidden>> escribió:

> > Isaac.
> >
> > Puedes colocar aquí ¿Por qué? se hace éste cambio por favor.
>
> El código que se esta proponiendo para merge ya existía en la revno
> 306, la cual crearba un path para escribir los archivos temporales y evitar
> problemas cuando el servidor de Openerp se levanta como demonio y hay
> problemas de permisos para acceder a la carpeta raiz. Sin embargo en ese
> mismo cambio se dejo de usar Superuser_id, generando problemas para
> imprimir el reporte PDF. En la revno 309 se revirtieron todos los cambios
> de la revno 306, lo cual no debió haber sido asi, solo se tenia que haber
> cambiado uid por Superuser_id.
>
> Por lo tanto, si revisamos el diff de la revno 306 con esta
> propuesta, la única diferencia que encontraremos es el cambio de uid por
> SUPERUSER_ID.
>
> > Lo técnicamente disponible está claro.
> >
> > Saludos.
> --
>
> https://code.launchpad.net/~openerp-mexico-maintainer/openerp-mexico-localization/7.0-ir_attachment_path_file_added-dev-yzk/+merge/190151
> You are reviewing the proposed merge of
> lp:~openerp-mexico-maintainer/openerp-mexico-localization/7.0-ir_attachment_path_file_added-dev-yzk
> into lp:openerp-mexico-localization/7.0.
>

--
--------------------
Saludos Cordiales

Nhomar G. Hernandez M.
[phone]
Skype: nhomar00
Web-Blog: http://geronimo.com.ve
Servicios IT: http://vauxoo.com
Linux-Counter: 467724
Correos:
<email address hidden>
<email address hidden>
twitter @nhomar

=== modified file 'l10n_mx_ir_attachment_facturae/ir_attachment_facturae.py'
--- l10n_mx_ir_attachment_facturae/ir_attachment_facturae.py	2013-10-08 21:32:07 +0000
+++ l10n_mx_ir_attachment_facturae/ir_attachment_facturae.py	2013-10-09 14:41:11 +0000
@@ -258,9 +258,12 @@
             invoice_obj = self.pool.get('account.invoice')
             type = self.browse(cr, uid, ids)[0].type
             wf_service = netsvc.LocalService("workflow")
+            (fileno, fname) = tempfile.mkstemp(
+                '.pdf', 'openerp_' + (invoice.fname_invoice or '') + '__facturae__')
+            os.close(fileno)
             report = invoice_obj.create_report(cr, SUPERUSER_ID, [invoice.id],
                                                "account.invoice.facturae.webkit",
-                                               invoice.fname_invoice)
+                                               fname)
             attachment_ids = attachment_obj.search(cr, uid, [
                 ('res_model', '=', 'account.invoice'),
                 ('res_id', '=', invoice.id),