id	summary	reporter	owner	description	type	status	component	version	severity	resolution	keywords	cc	stage	has_patch	needs_docs	needs_tests	needs_better_patch	easy	ui_ux
24886	Add process_lhs() method for Transform	Anssi Kääriäinen	Samriddha Kumar Tripathi	"The Lookup class has two methods, process_rhs() and process_lhs(). Typical usage is:
{{{
def as_sql(self, compiler, connection):
    lhs_sql, lhs_params = self.process_lhs(compiler, connection)
    rhs_sql, rhs_params = self.process_rhs(compiler, connection)
    # return the sql
}}}

The typical usage for transforms is:
{{{
def as_sql(self, compiler, connection):
    lhs_sql, lhs_params = compiler.compile(self.lhs)
    # return the sql
}}}

I think it could make sense to add process_lhs() to transform base class. This way one could write transforms using the same process_lhs() method that is used for Lookups, too. At least for me it is hard to remember when to use process_lhs() and when to use compiler.compile."	Cleanup/optimization	assigned	Database layer (models, ORM)	dev	Normal			Samriddha Kumar Tripathi	Accepted	1	0	0	0	0	0