File Changes for tdreszer

v301_base to v302_preview (2014-06-09 to 2014-06-16) v302

• src/hg/hgc/hgc.c
  • lines changed 21, context: html, text, full: html, text
    Extended URL trackDb setting to new tokens for item start and stop. Currently coded to use 1 bases coordinates for both start and stop. Redmine #13218
• src/hg/htdocs/goldenPath/help/trackDb/trackDbLibrary.shtml
  • lines changed 1, context: html, text, full: html, text
    Track name should not have dashes in it, because SQL table names do not allow it. Can't think of a reason that hub tracks (which are file-names) couldn't have dashes, but consistency is simplicity.
• src/hg/js/ajax.js
  • lines changed 56, context: html, text, full: html, text
    Polished for jshint. Some minor reformatting included.
  • lines changed 5, context: html, text, full: html, text
    Fixed setTimeouts (thanks Angie) and marked some code as possibly DEAD.
• src/hg/js/alleles.js
  • lines changed 44, context: html, text, full: html, text
    Polished for jshint.
  • lines changed 4, context: html, text, full: html, text
    Fixed setTimeouts (thanks Angie) and marked some code as possibly DEAD.
• src/hg/js/ddcl.js
  • lines changed 245, context: html, text, full: html, text
    Polished for jshint. Minor reformatting. Fixed atleast one bug.
  • lines changed 19, context: html, text, full: html, text
    Fixed setTimeout (thanks Angie) and retooled where the output of a bug that was fixed yesterday was relied upon downstream.
• src/hg/js/hgHubConnect.js
  • lines changed 34, context: html, text, full: html, text
    Had to merge in change from master, and clean up a bit.
• src/hg/js/hgTracks.js
  • lines changed 347, context: html, text, full: html, text
    Polished for jshint, which uncovered a couple minor bugs. Minimal reformatting. Plan to check in a reformatted version of the file soon.
  • lines changed 206, context: html, text, full: html, text
    Formatting changes to be laid on top of jshint polishing.
  • lines changed 1, context: html, text, full: html, text
    Fix for the (non-jshint related) bug where hiding then revisualizing a track without an intervening full page reload resultsin in an error. Redmine #13165.
  • lines changed 8, context: html, text, full: html, text
    Fix bug where gene suggest was failing to make UCSC genes visible. Redmine #13304.
  • lines changed 5, context: html, text, full: html, text
    Extended last fix to a couple more places that were relying upon creating hidden inputs, where adding to the queued cart vars makes more sense. Related to but not exactly redmine #13304.
  • lines changed 6, context: html, text, full: html, text
    Tightened the screws on several descrete cases where comparison to undefined might have been acting on undeclared variable. I have looked through all the js files I altered for jshint and only found these few cases which might (but in practice probably don't) trigger exceptions.
  • lines changed 6, context: html, text, full: html, text
    Backing out several of the previous 'tightening screws' changes. In particular typeof(thing.var) !== 'undefined' if thing.var === null. I thought I tested this eariler, but now it is breaking sortTable, as reported by QA.
• src/hg/js/hui.js
  • lines changed 286, context: html, text, full: html, text
    Plished for jshint and some reformatting. At least one bug corrected.
  • lines changed 1, context: html, text, full: html, text
    Fix for tool-tip hover on hgTrackUi matrices, as reported by Jonathon.
• src/hg/js/jquery.tablednd.js
  • lines changed 12, context: html, text, full: html, text
    Polished only UCSC changes for jshint. These 3rd party files modified by UCSC do NOT pass jshint. But changes are kept to minimum for later disambiguating.
• src/hg/js/makefile
  • lines changed 33, context: html, text, full: html, text
    All UCSC js files should pass jshint as part of make. Do not rely upon jshint.exclude.
• src/hg/js/subCfg.js
  • lines changed 148, context: html, text, full: html, text
    Polished for jshint. Minor reformatting.
• src/hg/js/ui.dropdownchecklist.js
  • lines changed 6, context: html, text, full: html, text
    Polished only UCSC changes for jshint. These 3rd party files modified by UCSC do NOT pass jshint. But changes are kept to minimum for later disambiguating.
• src/hg/js/utils.js
  • lines changed 591, context: html, text, full: html, text
    Polished for jshint. Minor formatting (e.g. all lines within 100). Strengthened normed(). Marked some code as possibly DEAD.
  • lines changed 35, context: html, text, full: html, text
    Changes at Angie's suggestion during pre-code-review. Redmine #13255.
  • lines changed 5, context: html, text, full: html, text
    Undeclared variables do not pass the 'truthiness' test without error. Redmine #13385.
  • lines changed 28, context: html, text, full: html, text
    Tightened the screws on several descrete cases where comparison to undefined might have been acting on undeclared variable. I have looked through all the js files I altered for jshint and only found these few cases which might (but in practice probably don't) trigger exceptions.
  • lines changed 12, context: html, text, full: html, text
    Backing out several of the previous 'tightening screws' changes. In particular typeof(thing.var) !== 'undefined' if thing.var === null. I thought I tested this eariler, but now it is breaking sortTable, as reported by QA.