Skip to:
Content
Pages
Categories
Search
Top
Bottom

New “Skeleton Component” for developers

Viewing 9 replies - 1 through 9 (of 9 total)
  • Avatar of Nicola Greco
    nicolagreco
    Participant

    @nicolagreco

    AWESOME!

    Avatar of Erwin Gerrits
    Erwin Gerrits
    Participant

    @egerrits

    Don’t get me wrong–I really appreciate this. It’s great to be able to start out with a skeleton component.

    That being said, it would be a better idea to name the component something else rather than “component”. Throughout all the files, we’re supposed to replace “component” with our own component name, however, there are instances such as the flag “current-component” that should not be renamed, creating confusion. Now we don’t know if we’re supposed to replace “component_id” or not.

    It would have been better to name the thing “example”, or another word not currently used in the bp structure so we can see exactly see where the component name comes in.

    I know the skeleton is created for us to learn the structure, I am just saying naming it “component” is confusing.

    Also, the skeleton component does not work straight uploaded into the mu-plugins? It adds a nav menu options, but the screens do not work, nor the sub navs. Any ideas?

    Avatar of Andy Peatling
    Andy Peatling
    Keymaster

    @apeatling

    Point taken, I may rename the functions.

    It will only work the trunk, it won’t work with beta 2.

    Avatar of Erwin Gerrits
    Erwin Gerrits
    Participant

    @egerrits

    Andy,

    Thanks for all your hard work! We all appreciate it.

    I am looking forward to the next release!

    Avatar of Andy Peatling
    Andy Peatling
    Keymaster

    @apeatling

    I’ve updated it to v1.1 with a better naming convention and added nonce security:

    http://tr.im/bpskel1_1 (zip link)

    Avatar of Erwin Gerrits
    Erwin Gerrits
    Participant

    @egerrits

    Thanks so much!

    Avatar of Erwin Gerrits
    Erwin Gerrits
    Participant

    @egerrits

    In the file “bp-example.php” on line 209:

    if ( $bp->current_example== $bp->example->slug ) {

    should read:

    if ( $bp->current_component== $bp->example->slug ) {

    shouldn’t it?

    Avatar of Andy Peatling
    Andy Peatling
    Keymaster

    @apeatling

    yep it should. I’ll update it this weekend.

    Avatar of Plrk
    plrk
    Participant

    @plrk

    The bad things about search&replace, huh? Throughout my plugin, there are comments like “In this mypluginname, we will…”

Viewing 9 replies - 1 through 9 (of 9 total)

The topic ‘New “Skeleton Component” for developers’ is closed to new replies.