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

PB-812 Naprawić nieprawidłowo wyświetlającą się cenę #37

Merged
merged 2 commits into from
Nov 27, 2023

Conversation

mbwawpl
Copy link
Collaborator

@mbwawpl mbwawpl commented Nov 23, 2023

No description provided.

{
$sales_enabled_key_exists = array_key_exists('sales_enabled', $price);

if (!$sales_enabled_key_exists) {
Copy link
Owner

Choose a reason for hiding this comment

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

jeśli nie znajduje tego klucza to zwraca true? czyli jesli nie znajduje to jest enabled? dziwnie się to czyta...

Copy link
Collaborator Author

@mbwawpl mbwawpl Nov 23, 2023

Choose a reason for hiding this comment

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

Tak, ta logika jest skopiowana z is_variant_sales_enabled w wolverine/product/Repository.php. Pewnie to jest potrzebne dla starszych danych.

Copy link
Owner

Choose a reason for hiding this comment

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

no tam jest babol... przydałoby sie to wyjaśnić tu (i tam) opisowo.... ze brak klucza = enabled, by było to jasne jakoś

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

W jakim sensie babol? Że to powinno działać inaczej, czy że niejasne jest, dlaczego tak działa jak teraz? Bo gdy dawniej nie było aktywowania sprzedaży na poziomie wariantu tylko kursu, to musimy przy braku tego ustawienia taki default przyjmować, że jest aktywny.

Copy link
Owner

Choose a reason for hiding this comment

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

Że jet nieczytelne / niejasne.

gdy dawniej nie było aktywowania sprzedaży na poziomie wariantu tylko kursu, to musimy przy braku tego ustawienia taki default przyjmować, że jest aktywny.

no i to jest spoko - a da sie kod napisać tak by to nieco zawierał? wyciągając to do metody priv czy coś? (ale nie dając komentarz :))

@mbwawpl mbwawpl merged commit cdc615f into master Nov 27, 2023
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

Successfully merging this pull request may close these issues.

3 participants