an error raise would be better since that's not a documentation note anyone 
would notice.

there seems to be a more general issue that you can put any SQL elements in 
literal() and that should not be happening in 1.4, so lets make a real bug

https://github.com/sqlalchemy/sqlalchemy/issues/5639



On Fri, Oct 9, 2020, at 4:49 AM, Massimiliano della Rovere wrote:
> Greetings,
> probably it's obvious to everybody but me, but I think it would be useful 
> specifying in both the literal and the bindparam documentation that 
> literal(bindparam(...)) is an invalid construct in sqlalchemy
> 

> --
> SQLAlchemy - 
> The Python SQL Toolkit and Object Relational Mapper
>  
> http://www.sqlalchemy.org/
>  
> To post example code, please provide an MCVE: Minimal, Complete, and 
> Verifiable Example. See http://stackoverflow.com/help/mcve for a full 
> description.
> --- 
> You received this message because you are subscribed to the Google Groups 
> "sqlalchemy" group.
> To unsubscribe from this group and stop receiving emails from it, send an 
> email to sqlalchemy+unsubscr...@googlegroups.com.
> To view this discussion on the web visit 
> https://groups.google.com/d/msgid/sqlalchemy/CADKhPGRrvZp9nkfojh4AYj9UwD8RTMGZfr1RpMFKkSC6zJnZwA%40mail.gmail.com
>  
> <https://groups.google.com/d/msgid/sqlalchemy/CADKhPGRrvZp9nkfojh4AYj9UwD8RTMGZfr1RpMFKkSC6zJnZwA%40mail.gmail.com?utm_medium=email&utm_source=footer>.

-- 
SQLAlchemy - 
The Python SQL Toolkit and Object Relational Mapper

http://www.sqlalchemy.org/

To post example code, please provide an MCVE: Minimal, Complete, and Verifiable 
Example.  See  http://stackoverflow.com/help/mcve for a full description.
--- 
You received this message because you are subscribed to the Google Groups 
"sqlalchemy" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to sqlalchemy+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/sqlalchemy/5176cc6d-2f76-4f6b-a4f0-b68feac9b227%40www.fastmail.com.

Reply via email to