multiple namespace/schema support about jugglingdb

[1602]

I think this diff will implement required feature:

diff --git a/lib/adapters/postgres.js b/lib/adapters/postgres.js
index e3f848c..aff215c 100644
--- a/lib/adapters/postgres.js
+++ b/lib/adapters/postgres.js
@@ -196,7 +196,7 @@ PG.prototype.fromDatabase = function (model, data) {
 };

 PG.prototype.escapeName = function (name) {
-    return '"' + name + '"';
+    return '"' + name.replace(/\./g, '"."') + '"';
 };

When defining model, just pass 'tableName' param, or use setTableName('database.table') in railwayjs

[fsateler]

The only problem I see is if a table really had a . In its name. I don't,
but maye someone will have one at some point. I'm not sure if that casee
warrants looking for a more elaborate solution

On 12 Apr 2012 20:26, "Anatoliy Chakkaev" <
[email protected]>
wrote:

I think this diff will implement required feature:

diff --git a/lib/adapters/postgres.js b/lib/adapters/postgres.js
index e3f848c..aff215c 100644
--- a/lib/adapters/postgres.js
+++ b/lib/adapters/postgres.js
@@ -196,7 +196,7 @@ PG.prototype.fromDatabase = function (model, data) {
 };

 PG.prototype.escapeName = function (name) {
-    return '"' + name + '"';
+    return '"' + name.replace(/\./g, '"."') + '"';
 };

When defining model, just pass 'tableName' param, or use
setTableName('database.table') in railwayjs

---

Reply to this email directly or view it on GitHub:
#68 (comment)

[1602]

Yep, this may cause issues, but we don't break existing code, since
setTableName is not documented feature yet.
And one more thing need to review - migration on postgres after this change.

On Fri, Apr 13, 2012 at 4:39 AM, fsateler <
[email protected]

wrote:

The only problem I see is if a table really had a . In its name. I don't,
but maye someone will have one at some point. I'm not sure if that casee
warrants looking for a more elaborate solution

On 12 Apr 2012 20:26, "Anatoliy Chakkaev" <
[email protected]>
wrote:

I think this diff will implement required feature:

diff --git a/lib/adapters/postgres.js b/lib/adapters/postgres.js
index e3f848c..aff215c 100644
--- a/lib/adapters/postgres.js
+++ b/lib/adapters/postgres.js
@@ -196,7 +196,7 @@ PG.prototype.fromDatabase = function (model, data) {
 };

 PG.prototype.escapeName = function (name) {
-    return '"' + name + '"';
+    return '"' + name.replace(/\./g, '"."') + '"';
 };

When defining model, just pass 'tableName' param, or use
setTableName('database.table') in railwayjs

---

Reply to this email directly or view it on GitHub:
#68 (comment)

---

Reply to this email directly or view it on GitHub:
#68 (comment)

Thanks,
Anatoliy Chakkaev

[1602]

Everything is working fine.