Olap question

Question

hi all,
whenver i execute the below query in olap(excel 2007), it gives error "sql command not ended properly"
i am completely new to olap,:( and i have to generate a report based on this
Private Sub Calendar1_Click()
End Sub
Private Sub Calendar2_Click()
Sql = "   SELECT employee_number,act_hour,floor(tot/60)||':'||mod(tot,60) tothours  FROM (SELECT employee_number,   "
Sql = Sql & "    act_hour   ,  "
Sql = Sql & "  SUM(to_number(regexp_substr(act_hour,'[^:]+',1,1))*60+to_number(regexp_substr(act_hour,'[^:]+',1,2))) over (partition BY employee_number) tot "
Sql = Sql & " FROM nap_PUNCH_DATA_EMP_LIST trn  WHERE DATE_ENTRAY between '23-AUG-2009' and  '24-AUG-2009');"
Sql = Sql & "and   DATE_ENTRAY between '%1' and  '%2' "
Sql = Replace(Sql, "%1", Format(Calendar1.Value, "DD-MMM-YYYY"))
Sql = Replace(Sql, "%2", Format(Calendar2.Value, "DD-MMM-YYYY"))
ActiveSheet.PivotTables(1).PivotCache.CommandText = Sql
Range("B16").Select
ActiveWorkbook.ShowPivotTableFieldList = False
End Subthanking in advance ,any help will be appreciated,please let me know if i have to post more details on this
regards
oracle user

Answer 1

Remove semicolon (;)
And you'll get another error:
This is your query:
SELECT employee_number, act_hour, floor(tot/60)||':'||mod(tot,60) tothours 
  FROM (
    SELECT employee_number, act_hour, SUM(
                                        to_number(
                                              regexp_substr(act_hour,'[^:]+',1,1)
                                                  )*60 +
                                        to_number(
                                              regexp_substr(act_hour,'[^:]+',1,2)
                                         ) over (partition BY employee_number) tot
      FROM nap_PUNCH_DATA_EMP_LIST trn 
     WHERE DATE_ENTRAY between '23-AUG-2009' and  '24-AUG-2009')
and   DATE_ENTRAY between '%1' and  '%2' In the last row you should replace AND with WHERE...
Max
[My Italian Oracle blog|http://oracleitalia.wordpress.com/2010/01/02/query-gerarchiche/]
Edited by: Massimo Ruocchio on Jan 5, 2010 10:17 AM
Added next error