Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement endpoint for document cloning #54

Closed
teosarca opened this issue Nov 22, 2016 · 8 comments
Closed

Implement endpoint for document cloning #54

teosarca opened this issue Nov 22, 2016 · 8 comments

Comments

@teosarca
Copy link
Member

aka the Copy with details from our Swing client.

@teosarca
Copy link
Member Author

teosarca commented Nov 22, 2016

teosarca added a commit to metasfresh/metasfresh that referenced this issue May 27, 2017
teosarca added a commit to metasfresh/metasfresh that referenced this issue May 29, 2017
@metas-mk metas-mk added this to the 2017-25 milestone Jun 6, 2017
@metas-mk metas-mk modified the milestones: 2017-30, 2017-25 Jul 22, 2017
@teosarca
Copy link
Member Author

Note to IT:

  • to test the actual task, pls wait for the frontend task
  • besides that, regression is uber important, i.e. copy with details shall work for:
    ** order (sales, purchase)
    ** invoice (sales, purchase)
    ** BOM
    ** Role
    ** GL Journal/Batch

metas-ts added a commit to metasfresh/metasfresh that referenced this issue Sep 1, 2017
[#553](metasfresh/metasfresh-webui-api-legacy#553)
webui notification-test fails
[#1157](metasfresh/metasfresh-webui-frontend-legacy#1157)
Delete User Notifications
[#2145](#2145) Partial
Internal usage action in Handling Unit Editor
 [#529](#529) Possibility
to easily create more than 1 referenced Doc per Invoice
 [#557](metasfresh/metasfresh-webui-api-legacy#557)
Picking Window: Open HU Selection window filtering
 [#560](metasfresh/metasfresh-webui-api-legacy#560)
Picking: don't allow picking included rows
[#556](metasfresh/metasfresh-webui-api-legacy#556)
Picking Window Main View adjustments
[#551](metasfresh/metasfresh-webui-api-legacy#551)
verify if lookup value is still valid
[#2237](#2237) Order
Control Report Attribute missing
[#528](metasfresh/metasfresh-webui-api-legacy#528)
Exception when opening PP Order issue / receipt
[#2102](#2102) maven
change "local" version from 1.0.0 to 10.0.0
[#1876](#1876) New CU
Labels
[#1142](metasfresh/metasfresh-webui-frontend-legacy#1142)
issue panel closes right after you issue a product
[#54](metasfresh/metasfresh-webui-api-legacy#54)
Implement endpoint for document cloning
[#549](metasfresh/metasfresh-webui-api-legacy#549)
Validcombination Window does not allow NEW in WebUI
[#2245](#2245) Replace
description field by memo in B2C
[#2258](#2258) add due
date to request grid view
[#2247](#2247) Make
Request Type a search field
[#2260](#2260) SQL -
problem to rollout master build to DB with issue-build

me-45
teosarca added a commit to metasfresh/metasfresh that referenced this issue Sep 7, 2017
teosarca added a commit to metasfresh/metasfresh that referenced this issue Sep 7, 2017
teosarca added a commit that referenced this issue Sep 7, 2017
solves issues like C_Order.PreparationDate was not copied

#54
teosarca added a commit that referenced this issue Sep 7, 2017
@teosarca teosarca self-assigned this Sep 7, 2017
@teosarca
Copy link
Member Author

teosarca commented Sep 8, 2017

(moved from metasfresh/metasfresh-webui-frontend-legacy#1164 (comment))

IT
I. Sales order: OK

  1. create a new sales order, add all mandatory fields
  2. change all the dates, so it won't be today
  3. set a reference, a dropshipment bpartner
  4. add 2 products, set an attribute for one and for the other one let it empty, set HU for one and the other one "no packing item", change the price for one and the other one leave it like it is
  5. complete the order
  6. go to actions, select Clone
    => new sales order opens on the same tab
    => status draft, with 2 lines of product (no TU line)
    => all the fields were copied (bpartner, products, attributes, dropship, packages) besides: dates (all set on today), reference (empty), price (was taken from pricelist)
  7. complete the order => TU line is added

II. purchase order OK??

  1. create a new purchase order, add all mandatory fields
  2. change all the dates, so it won't be today, also add tageslot
  3. set a reference, a dropshipment bpartner
  4. add 2 products, set an attribute for one and for the other one let it empty, set HU for one and the other one "no packing item", change the price for one and the other one leave it like it is
  5. complete the order
  6. go to actions, select Clone
    => new sales order opens on the same tab
    => status draft, with 2 lines of product (no TU line)
    => all the fields were copied (bpartner, products, attributes, dropship, packages, tageslot) besides: dates (all set on today, availability was empty), reference (empty), price (was taken from pricelist)
  7. complete the order => TU line is added
    .
    .question: Availability Date was not copied.. while on swing was. OK??

availabilitydate

III. sales invoice NOK?

  1. create a sales invoice for a sales order (with multiple lines, product and TU)
  2. clone this invoce
    => all fields are copied as they are (including date, reference)
    => document is draft
    => lines can change order (products first, TUs after) and they keep their prices
  3. create a new sales invoice manually, add a product line and change the price
  4. clone it
    => all fields are copied as they are (including date, reference)
    => document is draft
    => line doesn't keep it's price (on swing it does) NOK??

price

IV. purchase invoice

  1. create a purchase invoice for a purchase order (with multiple lines, product and TU)
  2. clone this invoce
    => all fields are copied as they are (including date, reference)
    => document is draft
    => lines can change order (products first, TUs after) and they keep their prices
  3. create a new purchase invoice manually, add a product line and change the price
  4. clone it
    => all fields are copied as they are (including date, reference)
    => document is draft
    => line keeps it's price

About step 2: i couldn't reproduce on swing an invoice creation with TUs first and products after

V. BOM NOK (probably it's related with the fact that you have to add the document no manually)
issue: #588

Error: org.postgresql.util.PSQLException: ERROR: duplicate key...
Server error
org.postgresql.util.PSQLException: ERROR: duplicate key value violates unique constraint "ppproductbomunique" Detail: Key (ad_client_id, value)=(1000000, P002737_AB Alicesalat 250g) already exists. SQL: INSERT INTO PP_Product_BOM (AD_Client_ID,AD_Org_ID,BOMType,BOMUse,CopyFrom,Created,CreatedBy,C_UOM_ID,DocumentNo,IsActive,M_Product_ID,Name,PP_Product_BOM_ID,Processing,Updated,UpdatedBy,ValidFrom,Value) VALUES (1000000,1000000,'A','M','N',TO_TIMESTAMP('2017-09-07 08:44:11','YYYY-MM-DD HH24:MI:SS'),2188223,100,'1000373','Y',2005577,'P002737_AB Alicesalat 250g',nextval('pp_product_bom_seq'),'N',TO_TIMESTAMP('2017-09-07 08:44:11','YYYY-MM-DD HH24:MI:SS'),2188223,TO_TIMESTAMP('2015-07-14','YYYY-MM-DD'),'P002737_AB Alicesalat 250g') RETURNING PP_Product_BOM_ID

VI. Role NOK (seems it's trying to copy the name)

Error: Der Rollenname muss eindeutig...
Server error
Der Rollenname muss eindeutig sein.

VII. GL Journal/Batch

Error: Cannot get TableName for...
Server error
Cannot get TableName for a null model. Possible development issue.

@teosarca
Copy link
Member Author

teosarca commented Sep 8, 2017

update: "II. purchase order OK??" was fixed.
The others are still pending.

@metas-ts
Copy link
Member

metas-ts commented Sep 8, 2017

not needed in branch 2017-09-12as talked with @metas-mk

@teosarca
Copy link
Member Author

VI. Role NOK

fixed

@teosarca
Copy link
Member Author

III. sales invoice NOK?
=> line doesn't keep it's price (on swing it does) NOK??

hmm checked, and in my case it worked :)

My test case was:

  • create a new sales invoice (manually)

  • batch entry: add Topinambour . In my price list it's sold with 1EUR

  • changed Topinambour's price from 1EUR to 2EUR

  • batch entry: add Topinambour. It was added with the price 2EUR

  • so now i have one line with price 1EUR and one line 2EUR

  • clone it

  • document looks nice :)

  • on the second line, changed the price (PriceEntered) from 2EUR to 3EUR

  • clone it

  • document looks nice :)

image

@metas-rc
Copy link
Member

Moving this task to done because all its parts are either covered or with followups.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants