Fix bug where onQuery() was only called the first time the container was mounted. Updated readme.

Author: Rick

README.md

@@ -10,8 +10,9 @@ Inspired by: [Building the Facebook Newsfeed with Relay](http://facebook.github.
 
 ## Features
 
-- Official Relay-inspired API and syntax.
-- Write declarative queries right in components as Promises.
+- Implements the official Relay API methods.
+- HOC: Higher-order component syntax just like Relay.
+- Write declarative queries as Promises.
 - Isomorphic architecture supports server-side rendering.
 - Works with React 0.12 and 0.13, and React Native!
 
@@ -24,18 +25,23 @@ Inspired by: [Building the Facebook Newsfeed with Relay](http://facebook.github.
 ## Usage
 
 ````js
-import React from "react";
+import React    from "react";
 import Transmit from "react-transmit";
 
-const Newsfeed = React.createClass(...);
+// Simpl React component.
+const Newsfeed = React.createClass({
+	render () {
+		return this.props.stories.map((story) => <li>{story.content}</li>);
+	}
+});
 
+//  Higher-order Transmit component that will contain the above React component.
 export default Transmit.createContainer(Newsfeed, {
 	queryParams: {
 		count: 10
 	},
 	queries: {
 		stories (queryParams) {
-			// All Transmit queries return a Promise.
 			return Promise.all([
 				Story.getQuery("story")
 			]);

package.json

@@ -1,7 +1,7 @@
 {
   "name": "react-transmit",
   "description": "Relay-inspired library based on Promises instead of GraphQL.",
-  "version": "2.5.0",
+  "version": "2.5.1",
   "license": "BSD-3",
   "repository": {
     "type": "git",

src/example/Like.js

@@ -23,7 +23,7 @@ const Like = React.createClass({
 });
 
 /**
- *  Higher-Order Transmit component that will contain the above React component.
+ *  Higher-order Transmit component that will contain the above React component.
  */
 export default Transmit.createContainer(Like, {
 	/**

src/example/Main.js

@@ -17,6 +17,9 @@ const Main = React.createClass({
 			</InlineCss>
 		);
 	},
+	/**
+	 * This part is optional. It allows you to capture the query results.
+	 */
 	onQuery (promise) {
 		promise.then((queryResults) => {
 			console.log("Main.onQuery: ", queryResults);

src/example/Newsfeed.js

@@ -37,15 +37,15 @@ const Newsfeed = React.createClass({
 			nextStoryId:      this.props.queryParams.nextStoryId + 1
 		}).then((queryResults) => {
 			/**
-			 * This part is optional. It allows you to capture the quert results.
+			 * This part is optional. It allows you to capture the query results.
  			 */
 			console.log("Newsfeed.setQueryParams: ", queryResults);
 		});
 	}
 });
 
 /**
- *  Higher-Order Transmit component that will contain the above React component.
+ *  Higher-order Transmit component that will contain the above React component.
  */
 export default Transmit.createContainer(Newsfeed, {
 	/**

src/example/Story.js

@@ -38,7 +38,7 @@ const Story = React.createClass({
 });
 
 /**
- *  Higher-Order Transmit component that will contain the above React component.
+ *  Higher-order Transmit component that will contain the above React component.
  */
 export default Transmit.createContainer(Story, {
 	/**

src/lib/createContainer.js

@@ -73,17 +73,13 @@ module.exports = function (Component, options) {
 			this.currentParams = assign({}, Container.queryParams, externalQueryParams);
 
 			if (!this.hasQueryResults()) {
-				var promise = this.setQueryParams({});
-
-				if (this.props.onQuery) {
-					this.props.onQuery.call(this, promise);
-				}
+				this.setQueryParams({});
 			}
 		},
 		setQueryParams: function (nextParams, optionalQueryName) {
 			var _this = this;
 
-			return new Promise(function (resolve, reject) {
+			var promise = new Promise(function (resolve, reject) {
 				var props = _this.props || {};
 				var promise;
 
@@ -119,6 +115,12 @@ module.exports = function (Component, options) {
 
 				resolve(promise);
 			});
+
+			if (this.props.onQuery) {
+				this.props.onQuery.call(this, promise);
+			}
+
+			return promise;
 		},
 		/**
 		 * @returns {boolean} true if all queries have results.