Ticket #3188 (closed bug: fixed)

Opened 6 years ago

Last modified 6 years ago

prevent exposure of internal methods

Reported by: scott.gonzalez Owned by: scott.gonzalez
Priority: blocker Milestone: 1.6rc1
Component: ui.core Version: 1.5.2
Keywords: Cc:
Blocking: Blocked by:

Description

Internal plugin methods should not be callable using the plugin API.

Example:

$(el).plugin('internalMethod');

Change History

comment:1 Changed 6 years ago by scott.gonzalez

  • Status changed from new to accepted

This will be handled by the widget factory. Precede internal method names with an underscore to prevent exposure through the API.

comment:2 Changed 6 years ago by scott.gonzalez

The widget factory was modified to support this in r547.

comment:3 Changed 6 years ago by scott.gonzalez

  • Status changed from accepted to closed
  • Resolution set to fixed

There is a separate ticket for each plugin:

#3194 ui.accordion
#3195 ui.autocomplete
#3196 ui.colorpicker
#3197 ui.datepicker
#3198 ui.dialog
#3200 ui.draggable
#3201 ui.droppable
#3202 ui.magnifier
#3203 ui.progressbar
#3204 ui.resizable
#3205 ui.selectable
#3206 ui.slider
#3209 ui.sortable
#3207 ui.spinner
#3208 ui.tabs

comment:4 Changed 6 years ago by Cloudream

  • Status changed from closed to reopened
  • Resolution fixed deleted

After individual plugins fixed, we should fix init/plugins to _init/_plugins

comment:5 Changed 6 years ago by scott.gonzalez

Widget factory methods updated in r552.

comment:6 Changed 6 years ago by Cloudream

only mouse* in drag/drop/sort/select left.

comment:7 Changed 6 years ago by Cloudream

  • Status changed from reopened to closed
  • Resolution set to fixed

comment:8 Changed 6 years ago by scott.gonzalez

  • Status changed from closed to reopened
  • Resolution fixed deleted

mouse* in slider still needs to be updated.

comment:9 Changed 6 years ago by Cloudream

  • Status changed from reopened to closed
  • Resolution set to fixed

done and pass the test case.

comment:10 Changed 6 years ago by rdworth

  • Milestone changed from 1.6 to 1.6rc1
Note: See TracTickets for help on using tickets.