Skip to content
This repository has been archived by the owner on Nov 23, 2021. It is now read-only.

Add package quantity to the form item #19

Merged
merged 3 commits into from
Apr 22, 2021

Conversation

silvesterk
Copy link
Collaborator

No description provided.

* @SerializedName("custcol_rsm_package_quantity")
* @var int|null
*/
private $posloviJobPackageQuantity;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Woohoo, prvi slučaj namespace 🥇
Prvo mi je palo na pamet da li možda da PosloviSalesOrderItem extend-uje SalesOrderItem, ali to bi onda zahtevalo da postoji i PosloviSalesOrderForm koji extend-uje SalesOrderForm i onda apstrakcija k vragu..
Lično, sudeći po RSM nazivu, bliži mi je naziv property-ja packageQuantity a da u phpDoc stoji naznaka da se polje trenutno koristi kod Poslova (ko zna, možda ga još neko /zlo/upotrebi kasnije? :D )

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Da li ti je to pre-generički? @silvesterk

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nah možemo tako, mislim trenutno je zamišljeno da se koristi samo na poslovima, u pitanju je situacija gde želimo da naglasimo i pratimo u statistikama ako smo prodali npr: 1 x Paket od 15 Start Oglasa. Do sada smo to radili neispravno (jer nije postojao način da se uradi drugačije) pa je glasilo: 15 x Paket Start Oglasa.

E sada postoji verovatnoća da će tako nešto nekom drugom biznisu da bude potrebno u budućnosti. Ispravljam da bude generično a u komentaru ostavljam da to za sada samo poslovi koriste.

@nebkam nebkam merged commit 5ba8f81 into main Apr 22, 2021
@nebkam nebkam deleted the feature/add-item-package-quantity branch August 11, 2021 05:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants