Meta Box

MB Group "std" is not showing up in settings page

Support › MB Group › MB Group "std" is not showing up in settings pageResolved

Viewing 12 posts - 1 through 12 (of 12 total)
  • Author
    Posts
  • January 7, 2019 at 5:29 PM #12927
    RazibRazib
    Participant

    This is my code: https://ghostbin.com/paste/pknxo

    Thanks for you time.

    January 7, 2019 at 8:18 PM #12932
    zeeshan@wphood.com[email protected]
    Participant

    https://ghostbin.com/paste/9mnkv

    There is an error in your code. Use semi-colon in the last then check it.

    January 7, 2019 at 10:39 PM #12933
    RazibRazib
    Participant

    I have no PHP error.
    Please check my screenshot - http://prntscr.com/m437f9
    How it behave in settings page.

    https://ghostbin.com/paste/bbzwp

    January 8, 2019 at 5:38 PM #12952
    zeeshan@wphood.com[email protected]
    Participant

    The std settings is applied per sub-field. Please set std for each sub-field of the group.

    January 8, 2019 at 6:54 PM #12953
    RazibRazib
    Participant

    That means I can not add pre-defined group values, But it is required for me(list item with sortable).

    January 8, 2019 at 9:17 PM #12954
    RazibRazib
    Participant

    I've edited some code meta-box-group/class-rwmb-group-field.php
    Now default group value is saved.

    https://ghostbin.com/paste/w3go6

    January 9, 2019 at 3:46 PM #12963
    Anh TranAnh Tran
    Keymaster

    That's so great, Razib. I've just added that to the latest version 1.2.18 (along with other fixes as well).

    February 12, 2019 at 3:50 AM #13265
    Joe AndersonJoe Anderson
    Participant

    I still was unable to render std values in my groups. I have created another pull request with a modification I made. Please check it out when you get a chance.

    Thanks!

    February 13, 2019 at 10:59 AM #13287
    Anh TranAnh Tran
    Keymaster

    Hi Joe,

    Thanks for your contribution. Your code respect the std in child field, so it's good. However, it fails with Razib code above. I have commented on your PR. Please update it so I can merge.

    February 14, 2019 at 1:36 AM #13311
    Joe AndersonJoe Anderson
    Participant

    Thanks. I believe I may need to get some more background on how the current functionality is setup as I don't believe I'm fully understanding how the root std field is meant to work.

    I am mainly trying to get the std field within the child fields to work for every instance of the group object (initial and clones).

    I will continue conversation on this within the merge request, unless you'd rather it be continued here.

    Thanks!

    February 14, 2019 at 2:43 PM #13322
    Anh TranAnh Tran
    Keymaster

    Let's discuss there as it's very code-specific.

    February 15, 2019 at 9:31 AM #13354
    Anh TranAnh Tran
    Keymaster

    I've just updated the PR and merged them. New version is also released. Please check.

  • Author
    Posts
Viewing 12 posts - 1 through 12 (of 12 total)
  • You must be logged in to reply to this topic.