Ваши занятия во множественном числе должны быть переименованы в единственное число
Кратность композиции от Order до OrderDetail необходимо изменить на Order <*>-1---------*-> OrderDetail
Кажется логичным иметь отношение от OrderDetail к Product , каждый OrderDetail связан с одним Product , и вы удаляете ProductId & ProductName становится бесполезным. Не уверен, что промежуточный итог имеет интерес, может быть, я не очень хорошо понимаю, какова цель.
Вы можете иметь отношение от Заказ до Оплата , или, может быть, вы также можете ввести класс Счет ?