[FIX] account_payment_pro_receiptbook: fix where string#899
[FIX] account_payment_pro_receiptbook: fix where string#899cav-adhoc wants to merge 1 commit intoingadhoc:19.0from
Conversation
|
|
There was a problem hiding this comment.
Pull request overview
Este PR intenta corregir cadenas SQL WHERE añadiendo el prefijo de raw string (r""), pero este cambio es innecesario y puede resultar confuso.
Changes:
- Se añade el prefijo
r""a dos cadenas SQL WHERE que no contienen secuencias de escape
| else: | ||
| where_string, param = super()._get_last_sequence_domain(relaxed) | ||
| where_string += " AND receiptbook_id is Null " | ||
| where_string += r" AND receiptbook_id is Null " |
There was a problem hiding this comment.
El uso de raw strings (r"") es innecesario aquí por la misma razón que en la línea 17. Esta cadena SQL tampoco contiene secuencias de escape que requieran el prefijo raw string. Se recomienda usar comillas normales.
|
|
||
| if self.receiptbook_id and is_payment: | ||
| where_string = "WHERE receiptbook_id = %(receiptbook_id)s AND name != '/'" | ||
| where_string = r"WHERE receiptbook_id = %(receiptbook_id)s AND name != '/'" |
There was a problem hiding this comment.
El uso de raw strings (r"") es innecesario en estos casos. Las raw strings se utilizan para evitar la interpretación de secuencias de escape con backslash (como \n, \t, \, etc.), pero estas cadenas SQL no contienen ningún backslash.
La cadena contiene:
%(receiptbook_id)sque es un placeholder de formateo de Python (no una secuencia de escape)'/'que es una barra diagonal entre comillas simples (no hay backslash)
Usar raw strings cuando no son necesarias puede ser confuso, ya que sugiere la presencia de secuencias de escape cuando no las hay. Se recomienda usar comillas normales para estas cadenas SQL.
2 participants
No description provided.