Closed fredericalpers closed 1 year ago
@yeneastgate @dai-eastgate please prioritize this issue, so we can release it with v4.13
@fredericalpers Ok, I got it. We will check and fix it asap.
@fredericalpers I would like to confirm with you that the backward compatibility you mentioned means:
By the way, the "check for lowerCase for all strings" will be checked only in the "price on request" function, right?
@fredericalpers I would like to confirm with you that the backward compatibility you mentioned means:
- checkbox "Show price on request" activated & field "price on request" is not added to the field list => "price on request" feature works
- checkbox "Show price on request" deactivated & field "price on request" is added to the field list => "price on request" feature works
- checkbox "Show price on request" activated & field "price on request" is added to the field list => "price on request" feature works
- checkbox "Show price on request" deactivated & field "price on request" is not added to the field list => "price on request" feature doesn't work. Please let me know your opinion about this, thanks!
By the way, the "check for lowerCase for all strings" will be checked only in the "price on request" function, right?
yes, please go ahead and try to fix it. Thank you :)
Hi @fredericalpers, @andernath: (1) Price on request must be added back to the field list for the above query to work. See screenshot below. Backwards compatibility must also be given for all other possibilities if the field is also added to the field list. => Backwards compatibility is fixed for all other possibilities if the field is also added to the field list. (2) Please also implement a check for lowerCase for all strings. => We have redirected to check condition in another way so this logic will be removed. I also recorded a video, please watch this and let me know your opinion. Thanks! https://files.fm/u/79rdprv8n
Hi @fredericalpers, @andernath: (1) Price on request must be added back to the field list for the above query to work. See screenshot below. Backwards compatibility must also be given for all other possibilities if the field is also added to the field list. => Backwards compatibility is fixed for all other possibilities if the field is also added to the field list. (2) Please also implement a check for lowerCase for all strings. => We have redirected to check condition in another way so this logic will be removed. I also recorded a video, please watch this and let me know your opinion. Thanks! https://files.fm/u/79rdprv8n
Looks good! We will review it in depth on monday, thank you :)
Merge and ready for release.
Current state
(1) in #492 we removed price on request from the field list, this is now causing problems in customized templates so that price on request is not displayed at all or prices are partially visible.
In these templates we use the following line of code e.g.:
<?php if(strtolower($currentEstate['preisAufAnfrage']) === 'ja' || strtolower($currentEstate['preisAufAnfrage']) === 'yes' || strtolower($currentEstate['preisAufAnfrage']) === 'si') { ?>
(2) Also the following code does not check for lower or uppercases, which can cause problems as well:
private function
displayTextPriceOnRequest($recordModified, $field){ if($recordModified['preisAufAnfrage'] === __("Yes", "onoffice-for-wp-websites") && !empty($recordModified[$field])) { $recordModified[ $field ] = esc_html__('Price on request', 'onoffice-for-wp-websites'); } }
Desired state
(1) Price on request must be added back to the field list for the above query to work. See screenshot below. Backwards compatibility must also be given for all other possibilities if the field is also added to the field list.
(2) Please also implement a check for lowerCase for all strings.