Ticket #2529808 (closed defect)

Reporter


Charlie McMackin
Opened: 01/11/11
Last modified: 08/18/11
Status: closed
Type: defect
Resolution: fixed

Owner


Luke Smith
Target Release: 3.4.0
Priority: P3 (normal)
Summary: DataTableScroll plugin errors with DataTable + DataSource
Description:

This is a filing for this forum post. It has a link to an example on jsfiddle which uses 3.3.0pr3 but the problem persists in main. In short, a
DataTableScroll plugin doesn't seem to work with a DataTable which has DataTableDataSource also plugged in. It appears that in datatable-scroll-debug.js line 338 the widget assumes the data is already
loaded.

There is also a possibility that this is user error on my part, but so far it appears I'm one of the few in IRC and the forums that is testing out the DataTable and I haven't received any suggestions
on how I might approach this differently. If that is the case, may I request an example which uses the 2 plugins be included?

Type: defect Observed in Version: 3.3.0
Component: DataTable Severity: S3 (normal)
Assigned To: Luke Smith Target Release: 3.4.0
Location: Library Code Priority: P3 (normal)
Tags: datatable,datasource,datatablescroll Relates To:
Browsers: Chrome
URL: http://jsfiddle.net/tsuru/qyjw8/
Test Information:

Have a console to view the errors on the supplied link. For the jsfiddle link, try commenting out the DataTableScroll plugin line and pressing the Run button. "#datatable2" should render fine.

Change History

Luke Smith

YUI Contributor

Posted: 02/18/11

Luke Smith

YUI Contributor

Posted: 03/3/11
  • location changed to Library Code
  • milestone changed to 3.4.0
  • priority changed to P3 (normal)
  • status changed from assigned to accepted
  • version changed from development master to 3.3.0

I'll have a look at this for 3.4. If you're right, and it appears you are, this is obviously not ok.

Adrian Lanning

YUI Contributor

Posted: 05/17/11

I've created a patch for this issue - more information here:

http://yuilibrary.com/gallery/show/user-patch-2529808

It will be available on the CDN after the regular CDN update on 2011-05-18.

Luke Smith

YUI Contributor

Posted: 06/24/11

Ticket #2530067 was marked as a duplicate of this ticket.

Luke Smith

YUI Contributor

Posted: 07/18/11
  • completed changed from 0 to .25
  • estimated changed from 0 to .25
  • sprint changed to sprint 3
  • status changed from accepted to checkedin

Luke Smith

YUI Contributor

Posted: 07/18/11
  • completed changed from 0.25 to 0
  • milestone changed from 3.4.0
  • remaining changed from 0 to .25
  • sprint changed from sprint 3
  • status changed from checkedin to reopened

Updated the wrong ticket. There's still hope for this getting fixed in 3.4.0, but not today.

Luke Smith

YUI Contributor

Posted: 07/18/11
  • status changed from reopened to accepted

Luke Smith

YUI Contributor

Posted: 07/18/11
  • milestone changed to 3.4.0

Luke Smith

YUI Contributor

Posted: 07/27/11
  • completed changed from 0 to .25
  • remaining changed from 0.25 to 0
  • sprint changed to sprint 3
  • status changed from accepted to checkedin

Adrian Lanning

YUI Contributor

Posted: 07/27/11

Hi Luke,

I'm having trouble understanding how your patch addresses this issue. This one is pretty much solely due to the scroll plugin assuming there is already data in the datatable; throws an error when used with any async datasource. I didn't see any changes to datatable-scroll in your referenced github links above.

Maybe I missed something?

George

YUI Developer

Posted: 08/18/11
  • resolution changed to fixed
  • status changed from checkedin to closed