camptocamp / c2c-rd-addons

GNU Affero General Public License v3.0
15 stars 58 forks source link

Stock Weighing not working on Odoo 8 #20

Open singleroot opened 8 years ago

singleroot commented 8 years ago

Hello,

Tried to install in my Odoo 8 but not working, is complaining all the time about missing price_unit_coeff field and that only seems to me that a dependency is missing... can you guide me? and if not feasible then let me know. thanks

ferdiga commented 8 years ago

please try to remove this line ./mrp_view.xml:270:

the client using odoo 8 didn't need the module which was designed for 6.1

ferdiga commented 8 years ago

please try to remove this line ./mrp_view.xml:270:

the client using odoo 8 didn't need the module which was designed for 6.1

ferdiga commented 8 years ago

please try to remove this line ./mrp_view.xml:270:

the client using odoo 8 didn't need the module which was designed for 6.1

ferdiga commented 8 years ago

please try to remove this line ./mrp_view.xml:270:

the client using odoo 8 didn't need the module which was designed for 6.1

ferdiga commented 8 years ago

please try to remove this line ./mrp_view.xml:270:

the client using odoo 8 didn't need the module which was designed for 6.1

ferdiga commented 8 years ago

please try to remove this line ./mrp_view.xml:270:

the client using odoo 8 didn't need the module which was designed for 6.1

ferdiga commented 8 years ago

please try to remove this line ./mrp_view.xml:270:

the client using odoo 8 didn't need the module which was designed for 6.1

ferdiga commented 8 years ago

please try to remove this line ./mrp_view.xml:270:

the client using odoo 8 didn't need the module which was designed for 6.1

ferdiga commented 8 years ago

please try to remove this line ./mrp_view.xml:270:

the client using odoo 8 didn't need the module which was designed for 6.1

ferdiga commented 8 years ago

please try to remove this line ./mrp_view.xml:270:

the client using odoo 8 didn't need the module which was designed for 6.1

ferdiga commented 8 years ago

please try to remove this line ./mrp_view.xml:270:

the client using odoo 8 didn't need the module which was designed for 6.1

ferdiga commented 8 years ago

please try to remove this line ./mrp_view.xml:270:

the client using odoo 8 didn't need the module which was designed for 6.1

ferdiga commented 8 years ago

please try to remove this line ./mrp_view.xml:270:

the client using odoo 8 didn't need the module which was designed for 6.1

ferdiga commented 8 years ago

please try to remove this line ./mrp_view.xml:270:

the client using odoo 8 didn't need the module which was designed for 6.1

ferdiga commented 8 years ago

please try to remove this line ./mrp_view.xml:270:

the client using odoo 8 didn't need the module which was designed for 6.1

ferdiga commented 8 years ago

please try to remove this line ./mrp_view.xml:270:

the client using odoo 8 didn't need the module which was designed for 6.1

ferdiga commented 8 years ago

please try to remove this line ./mrp_view.xml:270:

the client using odoo 8 didn't need the module which was designed for 6.1

ferdiga commented 8 years ago

please try to remove this line ./mrp_view.xml:270:

the client using odoo 8 didn't need the module which was designed for 6.1

ferdiga commented 8 years ago

please try to remove this line ./mrp_view.xml:270:

the client using odoo 8 didn't need the module which was designed for 6.1

ferdiga commented 8 years ago

please try to remove this line ./mrp_view.xml:270:

the client using odoo 8 didn't need the module which was designed for 6.1

ferdiga commented 8 years ago

please try to remove this line ./mrp_view.xml:270:

the client using odoo 8 didn't need the module which was designed for 6.1

ferdiga commented 8 years ago

please try to remove this line ./mrp_view.xml:270:

the client using odoo 8 didn't need the module which was designed for 6.1

ferdiga commented 8 years ago

please try to remove this line ./mrp_view.xml:270:

the client using odoo 8 didn't need the module which was designed for 6.1

ferdiga commented 8 years ago

please try to remove this line ./mrp_view.xml:270:

the client using odoo 8 didn't need the module which was designed for 6.1

ferdiga commented 8 years ago

please try to remove this line ./mrp_view.xml:270:

the client using odoo 8 didn't need the module which was designed for 6.1

ferdiga commented 8 years ago

please try to remove this line ./mrp_view.xml:270:

the client using odoo 8 didn't need the module which was designed for 6.1

ferdiga commented 8 years ago

please try to remove this line ./mrp_view.xml:270:

the client using odoo 8 didn't need the module which was designed for 6.1

ferdiga commented 8 years ago

please try to remove this line ./mrp_view.xml:270:

<field name="price_unit_coeff" groups="c2c_stock_accounting.group_c2c_stock_accounting_accountant" readonly="1"/>

the client using odoo 8 didn't need the module which was designed for 6.1

singleroot commented 8 years ago

Thanks a bunch, actually I commented that line but the dependency to chricar_stock_care keeps calling that field (also commented there) and others... now this one...but this is when I realized lots of dependencies pending to be met.

El campo move_value_cost no existe

singleroot commented 8 years ago

I dont really need the stock to be moved at this stage.... I can write that later... just he record of the weight will do :)

ferdiga commented 8 years ago

yeh we heavily modified 6.1 to comply to accounting rules

please just comment the fields and let me know the results or make a pull request

singleroot commented 8 years ago

Thanks, I commented lot of fields and get to make it working to at least try.

It seems very nice, due the fields commented.. I believe somethings are now working as expected but seems pretty awesome.

Can you make truly portable to 8?

singleroot commented 8 years ago

Also noted when trying to move to draft state again, gives following error:

wf_service = netsvc.LocalService("workflow") NameError: global name 'netsvc' is not defined

ferdiga commented 8 years ago

please add from openerp import netsvc at the beginning of stock_weighing.py

(it's a blind guess - it might also be missing else where)

singleroot commented 8 years ago

Added and it worked. Thanks again....

Lots of review and rewrite need on Odoo 8. I'll take it as a base and thanks a bunch for an extraordinary job

ferdiga commented 8 years ago

great please make a pull request or send me the modified code

anyhow for 9 the code must be rewritten too

singleroot commented 8 years ago

Sorry, forgot to tell you about this error:

raise ValueError('External ID not found in the system: %s' % (xmlid))

ParseError: "External ID not found in the system: stock.action_picking_tree6" while parsing /usr/lib/python2.7/site-packages/openerp/addons/chricar_stock_weighing/stock_weighing_view.xml:171, near

I commented it but seems like a menu link is missing on my system :)

ferdiga commented 8 years ago

yes, that's because stock was completely rewritten for v8.