vgritsenko 2003/01/29 21:22:54 Modified: src/blocks/databases/java/org/apache/cocoon/acting/modular DatabaseAction.java DatabaseAddAction.java src/blocks/databases/java/org/apache/cocoon/components/modules/input CollectionMetaModule.java Log: cosmetic changes Revision Changes Path 1.6 +16 -25 xml-cocoon2/src/blocks/databases/java/org/apache/cocoon/acting/modular/DatabaseAction.java Index: DatabaseAction.java =================================================================== RCS file: /home/cvs/xml-cocoon2/src/blocks/databases/java/org/apache/cocoon/acting/modular/DatabaseAction.java,v retrieving revision 1.5 retrieving revision 1.6 diff -u -r1.5 -r1.6 --- DatabaseAction.java 20 Jan 2003 16:51:10 -0000 1.5 +++ DatabaseAction.java 30 Jan 2003 05:22:54 -0000 1.6 @@ -232,9 +232,9 @@ public void configure(Configuration conf) throws ConfigurationException { super.configure(conf); if (this.settings != null) { - this.defaultModeNames.put(MODE_OTHERS, (String) this.settings.get("input", inputHint)); - this.defaultModeNames.put(MODE_OUTPUT, (String) this.settings.get("output", outputHint)); - this.defaultModeNames.put(MODE_AUTOINCR, (String) this.settings.get("autoincrement", databaseHint)); + this.defaultModeNames.put(MODE_OTHERS, this.settings.get("input", inputHint)); + this.defaultModeNames.put(MODE_OUTPUT, this.settings.get("output", outputHint)); + this.defaultModeNames.put(MODE_AUTOINCR, this.settings.get("autoincrement", databaseHint)); this.pathSeparator = (String) this.settings.get("path-separator", this.pathSeparator); String tmp = (String) this.settings.get("first-row",null); if (tmp != null) { @@ -257,7 +257,6 @@ */ public void compose(ComponentManager manager) throws ComponentException { this.dbselector = (ComponentSelector) manager.lookup(DataSourceComponent.ROLE + "Selector"); - super.compose(manager); } @@ -317,11 +316,10 @@ } output.setAttribute( null, objectModel, key, value ); } catch (Exception e) { - if (getLogger().isWarnEnabled()) - getLogger() - .warn( "Could not select output mode " - + (String) outputMode - + ":" + e.getMessage() ); + if (getLogger().isWarnEnabled()) { + getLogger().warn("Could not select output mode " + + outputMode + ":" + e.getMessage()); + } } finally { if (outputSelector != null) { if (output != null) @@ -538,9 +536,6 @@ throws ConfigurationException { String setMode = null; - int setMaster = -1; - String setMastersMode = null; - boolean manyrows = false; int offset = ( isKey ? 0: set.noOfKeys); for ( int i = offset; i < conf.length + offset; i++ ) { @@ -565,7 +560,6 @@ setMode = set.columns[i].modeConf.getAttribute("set", null); } if ( setMode != null ) { - manyrows = true; set.columns[i].isSet = true; set.isSet = true; if ( setMode.equals("master") ) { @@ -671,7 +665,6 @@ // find tables to work with Configuration[] tables = conf.getChildren("table"); String tablesetname = param.getParameter("table-set", (String) this.settings.get("table-set")); - Map set_tables = null; // default to old behaviour Map modeTypes = null; @@ -746,11 +739,10 @@ } output.commit( null, objectModel ); } catch (Exception e) { - if (getLogger().isWarnEnabled()) - getLogger() - .warn( "Could not select output mode " - + (String) outputMode - + ":" + e.getMessage() ); + if (getLogger().isWarnEnabled()) { + getLogger().warn("Could not select output mode " + + outputMode + ":" + e.getMessage()); + } } finally { if (outputSelector != null) { if (output != null) @@ -779,11 +771,10 @@ } output.rollback( null, objectModel, e); } catch (Exception e2) { - if (getLogger().isWarnEnabled()) - getLogger() - .warn( "Could not select output mode " - + (String) outputMode - + ":" + e2.getMessage() ); + if (getLogger().isWarnEnabled()) { + getLogger().warn("Could not select output mode " + + outputMode + ":" + e2.getMessage()); + } } finally { if (outputSelector != null) { if (output != null) 1.3 +2 -4 xml-cocoon2/src/blocks/databases/java/org/apache/cocoon/acting/modular/DatabaseAddAction.java Index: DatabaseAddAction.java =================================================================== RCS file: /home/cvs/xml-cocoon2/src/blocks/databases/java/org/apache/cocoon/acting/modular/DatabaseAddAction.java,v retrieving revision 1.2 retrieving revision 1.3 diff -u -r1.2 -r1.3 --- DatabaseAddAction.java 7 Jan 2003 23:56:08 -0000 1.2 +++ DatabaseAddAction.java 30 Jan 2003 05:22:54 -0000 1.3 @@ -74,7 +74,6 @@ */ public class DatabaseAddAction extends DatabaseAction { - /** * set all necessary ?s and execute the query */ @@ -120,7 +119,7 @@ * @param statement the insert statement * @param objectModel the objectModel object * @param outputMode name of the requested output module - * @param result sitemap result object + * @param results sitemap result object * @return the number of columns by which to increment the currentIndex */ protected int setKeyAuto ( Configuration table, Column column, int currentIndex, int rowIndex, @@ -255,7 +254,6 @@ StringBuffer queryBuffer = new StringBuffer("INSERT INTO "); StringBuffer valueBuffer = new StringBuffer(") VALUES ("); - AutoIncrementModule dah; queryBuffer.append(table.getAttribute("name")); queryBuffer.append(" ("); 1.5 +3 -14 xml-cocoon2/src/blocks/databases/java/org/apache/cocoon/components/modules/input/CollectionMetaModule.java Index: CollectionMetaModule.java =================================================================== RCS file: /home/cvs/xml-cocoon2/src/blocks/databases/java/org/apache/cocoon/components/modules/input/CollectionMetaModule.java,v retrieving revision 1.4 retrieving revision 1.5 diff -u -r1.4 -r1.5 --- CollectionMetaModule.java 17 Dec 2002 14:53:58 -0000 1.4 +++ CollectionMetaModule.java 30 Jan 2003 05:22:54 -0000 1.5 @@ -86,7 +86,6 @@ protected Configuration memberConf = null; - public void configure(Configuration config) throws ConfigurationException { @@ -96,8 +95,6 @@ } - - public Object getAttribute( String name, Configuration modeConf, Map objectModel ) throws ConfigurationException { @@ -128,7 +125,6 @@ // in that case it replaces '*' otherwise it is // ignored String jType = mConf.getAttribute("type","string"); - SortedSet matchset = new TreeSet(); String pName = mConf.getAttribute("name"); int index = pName.indexOf("*"); if (index>-1) { @@ -158,10 +154,7 @@ } - - - - public Iterator getAttributeNames( Configuration modeConf, Map objectModel ) + public Iterator getAttributeNames( Configuration modeConf, Map objectModel ) throws ConfigurationException { if (!this.initialized) { @@ -187,7 +180,6 @@ } } - Iterator names = getNames(objectModel, this.input, this.defaultInput, this.inputConf, null, inputName, inputConfig); @@ -233,9 +225,7 @@ } - - - public Object[] getAttributeValues( String name, Configuration modeConf, Map objectModel ) + public Object[] getAttributeValues( String name, Configuration modeConf, Map objectModel ) throws ConfigurationException { Iterator names = this.getAttributeNames( modeConf, objectModel ); @@ -247,5 +237,4 @@ return values.toArray(); } - }
---------------------------------------------------------------------- In case of troubles, e-mail: [EMAIL PROTECTED] To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]