[Libreoffice-commits] core.git: offapi/com

[Andrzej J.R. Hunt]

 offapi/com/sun/star/sdbc/XColumnLocate.idl |   12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

New commits:
commit a042cdd7eff15ed5f6cbd613588b3fca9509a85e
Author: Andrzej J.R. Hunt <andrzej at ahunt.org>
Date:   Thu Sep 12 09:05:50 2013 +0100

    Update XColumnLocate to specify SQLException for invalid column.
    
    This is to reflect the JDBC specification where invalid column names
    result in an SQLException.
    
    (The drivers within LibreOffice are being updated to reflect this new
    specification.)
    
    Change-Id: I76cdf9d5d15d55b534b28219b541ff9190365f9d
    Reviewed-on: https://gerrit.libreoffice.org/5921
    Reviewed-by: Fridrich Strba <fridrich at documentfoundation.org>
    Tested-by: Fridrich Strba <fridrich at documentfoundation.org>

diff --git a/offapi/com/sun/star/sdbc/XColumnLocate.idl b/offapi/com/sun/star/sdbc/XColumnLocate.idl
index 0943898..a7c8d52 100644
--- a/offapi/com/sun/star/sdbc/XColumnLocate.idl
+++ b/offapi/com/sun/star/sdbc/XColumnLocate.idl
@@ -45,12 +45,22 @@ published interface XColumnLocate: com::sun::star::uno::XInterface
 {
 
     /** maps the given ResultSet column name to its ResultSet column index.
+
+        <p>
+        The specification before LibreOffice 4.2 left unspecified what should
+        happen for an invalid column name. As a result some drivers written
+        against the older speification may return a special invalid value, such
+        as a negative number, zero, or a number greater than the number of
+        columns.
+        </p>
+
         @param columnName
             the name of the column
         @returns
             the position of the column
         @throws SQLException
-                if a database access error occurs.
+                if the column named <code>columnName</code> does not exist,
+                or a database access error occurs.
      */
     long findColumn([in]string columnName) raises (SQLException);
 };