ericmckean / google-refine

Automatically exported from code.google.com/p/google-refine
Other
0 stars 0 forks source link

Typo in webapp/modules/core/scripts/reconcilition/recon-dialog.js #526

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Line 210:

elmts.typeInput.suggestT({ type: '/type/type' });

should be

elmts.typeInput.suggest({ type: '/type/type' });

?

Original issue reported on code.google.com by danpaulsmith on 31 Jan 2012 at 4:26

GoogleCodeExporter commented 9 years ago
I wonder if it came from SuggestTerm command extension at one time ? Here's 
some other areas that a search brings up: 
http://code.google.com/p/google-refine/source/search?q=suggestT&origq=suggestT&b
tnG=Search+Trunk

Original comment by thadguidry on 31 Jan 2012 at 4:50

GoogleCodeExporter commented 9 years ago
wow, the search is good - I'll use that in future. Seeing as there are so many 
suggestT's - I guess it's not a bug?

Original comment by danpaulsmith on 31 Jan 2012 at 4:56

GoogleCodeExporter commented 9 years ago
What problem are you having in the recon dialog specifically ?  that lead you 
to searching the JS code to begin with ?

Original comment by thadguidry on 31 Jan 2012 at 5:11

GoogleCodeExporter commented 9 years ago
I'm doing some development around reconciliation - lead me through that file 
and thus onto the supposed typo.

Original comment by danpaulsmith on 14 Feb 2012 at 2:59

GoogleCodeExporter commented 9 years ago
From the discussion, this doesn't sound like a bug to me.  Feel free to reopen 
if you believe otherwise.

Original comment by tfmorris on 13 Mar 2012 at 3:12