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

Material Dispo wrong Quantities after Material receipt #3935

Closed
metas-mk opened this issue Apr 25, 2018 · 3 comments
Closed

Material Dispo wrong Quantities after Material receipt #3935

metas-mk opened this issue Apr 25, 2018 · 3 comments

Comments

@metas-mk
Copy link
Member

metas-mk commented Apr 25, 2018

Is this a bug or feature request?

Bug

What is the current behavior?

After Material Receipt, the ATP Amount is set to the last received Quantity.

Which are the steps to reproduce?

Open, try and see.

What is the expected or desired behavior?

ATP shall not have the last Quantity, shall have the summed Quantity,

metas-ts added a commit that referenced this issue Apr 25, 2018
* M_Transactions's ASIs are not "reliable" for storage, so we now gather the attributes of our associated HUs
* M_Transaction deletes are now handled too
* also rename HUOnHandQtyChangeDescriptor to HUDescriptor since it now also contains a ProductDescriptor
* also minor improvements (javadoc, other cleanups) here and there

Material Dispo wrong Quantities after Material receipt #3935
@metas-ts
Copy link
Member

TODO

  • add M_HU_ID to MD_Candidate_Transaction_Detail

metas-ts added a commit that referenced this issue Apr 25, 2018
Material Dispo wrong Quantities after Material receipt #3935
metas-ts added a commit that referenced this issue Apr 27, 2018
Material Dispo wrong Quantities after Material receipt #3935
metas-ts added a commit that referenced this issue Apr 27, 2018
Material Dispo wrong Quantities after Material receipt #3935
metas-ts added a commit that referenced this issue Apr 27, 2018
#3935-app - Material Dispo wrong Quantities after Material receipt
@metas-rc metas-rc self-assigned this Apr 27, 2018
@metas-rc
Copy link
Member

Results of IT

Tested on webui

  • I created a new product because for the old ones all looked random.
  • I made a sales order with qty 100 for this product and a certain attribute A and I completed it
  • =>OK: The ATP turned -100
  • I did a purchase order with the same product and attribute and with the qty 200. I made a receipt out of it
  • => OK: The ATP turned 100

I did asome other Sales orders and Receipts as follows:

Sales Order -> qty 100 -> ATP -100 => OK
Purchase OrdeReceiptSales Order -> qty 1000 -> ATP -900 => OK
Receipt -> qty 760 -> ATP -140 => OK
Receipt -> qty 760 -> ATP 620 => OK
Sales Order -> qty 6200 -> ATP -3960 => NOK
Receipt -> qty 9000 -> ATP 5040 => OK
SalesOrder -> qty 5000 -> ATP = 20280 => NOK

As you can see, the receipt seems to change the atp correctly but the sales part breaks it after some itterations.

@metas-rc
Copy link
Member

Followup created: #3952

metas-ts added a commit that referenced this issue Apr 30, 2018
[#3960](#3960) Improve the speed of scripts applier
[#956](metasfresh/metasfresh-webui-api-legacy#956) Rest endpoint: delete all notifications
[#1756](metasfresh/metasfresh-webui-frontend-legacy#1756) [alt]++ and [alt]+q not working properly anymore
[#3925](#3925) Sales order: just-in-time creation of conditions for a product without conditions
[#3937](#3937) Create process for importing discount schema
[#3933](#3933) Credit limit: Credit used does not update when you reverse/void order
[#3946](#3946) Write M_ShipmentScheduleRecompute.AD_Pinstance_ID to queue-wp-log
[#3945](#3945) Get rid of "QtyReserverd Recalculate" AD_Scheduler_ID=550010
[#3869](#3869) Procurement - add AD_Issue_ID to I_PMM_QtyReport_Event, I_PMM_WeekReport_Event and I_PMM_RfQResponse_ChangeEvent
[#3301](#3301) Allow Batchbooking via Payment Selection
[#3889](#3889) Report for Pharma Permission/ Authorisation control
[#3800](#3800) Improve inventory import
[#3923](#3923) Change Picking Terminal columns and fix auto process picking
[#3935](#3935) Material Dispo wrong Quantities after Material receipt
[#3914](#3914) Avoid numeric values with too many trailing zeros
[#3891](#3891) Pharma: Don't allow recording of pricing conditions that are not allowed
[#1540](metasfresh/metasfresh-webui-frontend-legacy#1540) Shortcut Alt++ not working on non German Keyboard Layout
[#3929](#3929) Show BasePriceSystem in Document Line and Documents
[#949](metasfresh/metasfresh-webui-api-legacy#949) Business Partner Pharma Product Org
[#3927](#3927) Base Price + Surplus in Pricing Conditions not working
[#3928](#3928) Picking: Reactivate Picked HU/ Processed Picking
[#3880](#3880) ShipmentLines's TU-Qty is updated wrongly from schedule's QtyOrdered_Override
[#3910](#3910) Credit limit: treat credit status on NULL as it would be X (No check)
[#3875](#3875) Request notifications shall use new notifications API
[#3708](#3708) Send Mail too when notification
[#3837](#3837) MSV3 products ban list
[#945](metasfresh/metasfresh-webui-api-legacy#945) Business Partner Pharma Product Create New Issue
[#3909](#3909) docoutbound mailing: Autodect if MailWorkpackageProcessor.EmailMessage is HTML
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

3 participants