Fix failing if getApplicableNode() receive no node #21

Author: hudochenkov

CHANGELOG.md

@@ -2,6 +2,9 @@
 All notable changes to this project will be documented in this file.
 This project adheres to [Semantic Versioning](http://semver.org/).
 
+## 1.4.1
+* Fix issue with a rule content starting with a comment and follow by a rule. Error happens if config has any option except `sort-order`. #21
+
 ## 1.4.0
 * Added `preserve-empty-lines-between-children-rules`, which preserve empty lines between children rules and preserve empty lines for comments between children rules. #20
 

index.js

@@ -185,12 +185,14 @@ function getApplicableNode(lookFor, node) {
 	// find if there any rules before, and skip the comments
 	var prevNode = node.prev();
 
-	if (prevNode.type === lookFor) {
-		return node;
-	}
+	if (prevNode) {
+		if (prevNode.type === lookFor) {
+			return node;
+		}
 
-	if (prevNode.type === 'comment') {
-		return getApplicableNode(lookFor, prevNode);
+		if (prevNode.type === 'comment') {
+			return getApplicableNode(lookFor, prevNode);
+		}
 	}
 
 	return false;

package.json

@@ -1,6 +1,6 @@
 {
   "name": "postcss-sorting",
-  "version": "1.4.0",
+  "version": "1.4.1",
   "description": "PostCSS plugin to sort rules content with specified order.",
   "keywords": [
     "postcss",

test/fixtures/issue-21.css

@@ -0,0 +1,6 @@
+.form-subsection-tbl .tbl-name {
+	/* obscure stuff */
+	#page & {
+		width: 166px;
+	}
+}

test/fixtures/issue-21.expected.css

@@ -0,0 +1,6 @@
+.form-subsection-tbl .tbl-name {
+	/* obscure stuff */
+	#page & {
+		width: 166px;
+	}
+}

test/test.js

@@ -252,6 +252,12 @@ test('Should preserve empty lines between children rules and don\'t create unnee
 	});
 });
 
+test('Should not fail if getApplicableNode() receive no node. Issue #21', t => {
+	return run(t, 'issue-21', {
+		'empty-lines-between-children-rules': 1
+	});
+});
+
 // test('Should sort LESS files', t => {
 //     return run(t, 'less.less', {}, 'less');
 // });