!.niL? is better because .present? would return true if sql == ""