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

Merge Problem #54

Open
florian33 opened this issue Aug 28, 2013 · 4 comments
Open

Merge Problem #54

florian33 opened this issue Aug 28, 2013 · 4 comments

Comments

@florian33
Copy link

Hi,

We are expecting some troubles.
I'm using the merge module.
What's that currently doing, is deleting all the Vehicule Application.
I don't know if it's only my case.
I create a F-250 and F-250 Light Duty.
F-250 Light Duty 1990-2013 is applied for a product.
I want to merge both of them in F-250.
What I finally had is F-250 1999 for the product, others are lost.

Someone get the same trouble ?

@florian33
Copy link
Author

I tried with a fresh installation, here is what I've done :
1 - I create 1 make with 2 models and 4 years for bothh of them
Mazda Mazdaspeed 3 1980 to 1983
Mazda Ram 3 2030 to 2033

2 - I create a simple Product and assign all my vehicle
image

3 - Search in the frontend works well. So i try to merge Mazdaspeed 3 into Ram 3
image

4- it keeps years in Vehicle manage section
image

5- But doesn't keep Vehicule Application :
image

Hope that will help you !

@florian33
Copy link
Author

I tried this on test website :
Error 504 Gateway timeout and same problem
You can check it with SKu #Wizzotest1 on test website

@joshribakoff
Copy link
Member

The first post, step 3 looks like the correct behaviour, it merges the models but leaves the years as is. Instead I suppose you'd like it to have all the years from both vehicles? That actually makes sense, I guess I coded it in a confusing way. The second issue just shows that the operation took longer than your php.ini allowed.

A good workaround is to export your CSV, and edit the fitments there then re-import. But if you can elaborate on how the behavior should change, I can look into changing it. FYI a future version will have a faster merge operation, so the timeout is less of an issue. For now you can increase php.ini settings

@florian33
Copy link
Author

Yes, exactly , in step 4 we would like to see Ram 3 1980 - 1983 and Ram 3 2030-2033 if that's possible.

Thanks you.

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

No branches or pull requests

2 participants