leoric - npm Package Compare versions

Comparing version 0.1.1 to 0.1.2

History.md

@@ -0,1 +1,6 @@
+0.1.2 / 2017-12-14
+==================
+
+ * Copy left table's orders into subquery to make order/limit work when combined.
+
 0.1.1 / 2017-12-13
@@ -2,0 +7,0 @@ ==================

lib/spell.js

@@ -302,2 +302,12 @@ 'use strict'
 
+  subspell.orders = []
+  for (let i = orders.length - 1; i >= 0; i--) {
+    const [name, order] = orders[i]
+    const { type, qualifiers, value } = parseExpr(name)
+    if (type == 'id' && !qualifiers || qualifiers[0] == baseName) {
+      subspell.orders.unshift([value, order])
+      orders[i] = [`${baseName}.${value}`, order]
+    }
+  }
+
   const columns = []
@@ -320,3 +330,2 @@ for (const qualifier in selection) {
 
-  subspell.orders = []
   if (subspell.attributes.size > 0 || subspell.whereConditions.length > 0 || skip > 0 || rowCount > 0) {
@@ -323,0 +332,0 @@ chunks.push(`FROM (${formatSelectWithoutJoin(subspell)}) AS ${escapeId(baseName)}`)

package.json

 {
   "name": "leoric",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "description": "Object-relational mapping for Node.js",
@@ -5,0 +5,0 @@ "main": "index.js",

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

49419

increased by0.89%

Lines of code

1571

increased by0.51%

No dependency changes