Permalink
Browse files

add todos, docs

  • Loading branch information...
1 parent cd03b10 commit daa364be75a5ef7d7cbfd9f888976568085b24f5 @jpmckinney jpmckinney committed Feb 10, 2011
Showing with 5 additions and 1 deletion.
  1. +2 −1 core/AbstractManager.js
  2. +1 −0 helpers/jquery/ajaxsolr.theme.js
  3. +2 −0 widgets/jquery/PagerWidget.js
@@ -32,7 +32,8 @@ AjaxSolr.AbstractManager = AjaxSolr.Class.extend(
proxyUrl: null,
/**
- * The default Solr servlet.
+ * The default Solr servlet. You may prepend the servlet with a core if using
+ * multiple cores.
*
* @field
* @public
@@ -7,6 +7,7 @@
* @param {String} list The list to append items to.
* @param {Array} items The list of items to append to the list.
* @param {String} [separator] A string to add between the items.
+ * @todo Return HTML rather than modify the DOM directly.
*/
AjaxSolr.theme.prototype.list_items = function (list, items, separator) {
jQuery(list).empty();
@@ -10,6 +10,8 @@
* @expects this.target to be a list.
* @class PagerWidget
* @augments AjaxSolr.AbstractWidget
+ * @todo Don't use the manager to send the request. Request only the results,
+ * not the facets. Update only itself and the results widget.
*/
AjaxSolr.PagerWidget = AjaxSolr.AbstractWidget.extend(
/** @lends AjaxSolr.PagerWidget.prototype */

2 comments on commit daa364b

@epugh
epugh commented on daa364b Feb 24, 2011

I like seeing activity! Especially nice doco activity! I am updating the Solr book for 3.1 and referencing people to ajax-solr in preference to SolrJS ;-)

@jpmckinney
Contributor

Sweet :) I think SolrJS is gone anyway: http://www.solrstuff.org/

Please sign in to comment.