I hope I am not oversimplifying your problem but what you are saying is - Order can have many OrderEntry, and OrderEntry is described by a Type of "Good".
You refactored your model from:
Order -> Goods Type (* to *)
TO:
Order-->OrderEntry (1 to *)
OrderEntry --> Goods (* to 1) [As one Goods Type can describe multiple OrderEntry]
This is more or less similar to Figure 1 and Figure 2 as explained by Scott Ambler (see "Enrollment" association in link below) and seems to justify the original cardinality.
http://www.agilemodeling.com/artifacts/classDiagram.htm
Hope this helps.