На айтемы блока рекомендую ссылаться полностью - :имя_блока.имя_айтема.

Вы применяете конкатенацию значений :date1, :date2, :curr с телом default_where (фактически, с телом запроса к базе). Это в большинстве случаев будет приводить к лишним parse при выполнении. Да и бороться со случайными одиночным кавычками вам придется. Можно применять непосредственную ссылку на айтем блока внутри выражения default_where, т.е. например, не:
'tab.field_a = nvl('||:block1.field_a_search_criteria||', 100)'
а
'tab.field_a = nvl(:block1.field_a_search_criteria, 100)'
Forms, как бы, умный, сам найдет нужный айтем, возьмет его значение и отправит на сервер запрос с соответствующим bind.

В случае to_date для строкового литерала настоятельно рекомендуется указывать форматную маску.

Вызывает сомнение целесообразность (select min/max from base) в подзапросах, как минимум, с точки зрения производительности.

А юзерам будет удобно пользоваться методом ввода :curr через запятую? Может, дружественнее будет им чекбоксики расставить в желаемых валютах по списку?