ADempiere ERP Business Suite / Bugs / #1685 Port missing connect for Test.sql

#1685 Port missing connect for Test.sql

Milestone: Core

Status: closed-remind

Owner: Carlos Ruiz

Labels: Installation (44)

Priority: 3

Updated: 2010-03-31

Created: 2009-01-16

Creator: mcicchelli

Private: No

install/src/org/compiere/install/ConfigOracle.java
(Line 380)

// TNS Name Info via sqlplus
String sqlplus = "sqlplus system/" + systemPassword + "@"
+ databaseServer.getHostName()
+ "/" + databaseName
+ " @utils/oracle/Test.sql";

Should include port:
String sqlplus = "sqlplus system/" + systemPassword + "@"
+ "//" + databaseServer.getHostName()
+ ":" + databasePort
+ "/" + databaseName
+ " @utils/oracle/Test.sql";

Discussion

mcicchelli - 2009-01-16

assigned_to: nobody --> vpj-cd
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Carlos Ruiz - 2010-03-16

assigned_to: vpj-cd --> nobody

priority: 5 --> 3

status: open --> open-remind
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Carlos Ruiz - 2010-03-16

Still not solved in class - patch seems correct, it needs testing.

Workaround: use port 1521

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Enrique Ruibal - 2010-03-16

Hi Carlos,

This patch is working in production with one Customer, I think you can safely commit it in trunk.

The reason for adding the port number on this, is becasue some people would like to use a different port to listen to DB for security reasons.

Regards,

P.S. Thank for all your support to ADempiere Project

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Carlos Ruiz - 2010-03-16

assigned_to: nobody --> globalqss

status: open-remind --> pending-remind
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Carlos Ruiz - 2010-03-16

Thanks Enrique for confirming it works.

Committed revision 11679.
http://adempiere.svn.sourceforge.net/adempiere/?rev=11679&view=rev

Regards,

Carlos Ruiz

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Tony Snook - 2010-03-17

Merged 11679 to trunk in Revision: 11694
http://adempiere.svn.sourceforge.net/adempiere/?rev=11694&view=rev

Tony

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

SourceForge Robot - 2010-03-31

This Tracker item was closed automatically by the system. It was
previously set to a Pending status, and the original submitter
did not respond within 14 days (the time period specified by
the administrator of this Tracker).

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

SourceForge Robot - 2010-03-31

status: pending-remind --> closed-remind
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Log in to post a comment.