Skip to content

issues:710 - Hotfix on regex pattern and resolve problem to show list-item. and add menu select feature.#738

Open
vheidari wants to merge 2 commits intovalendesigns:masterfrom
vheidari:master
Open

issues:710 - Hotfix on regex pattern and resolve problem to show list-item. and add menu select feature.#738
vheidari wants to merge 2 commits intovalendesigns:masterfrom
vheidari:master

Conversation

@vheidari
Copy link

@vheidari vheidari commented Oct 2, 2021

  1. Hotfix on regex pattern and resolve problem to show list-item.
    Option setting not reset and list-item meta box not showing properly #714 For this situation we just needing to search for uppercase 'O' and 'C' letter in the serialized string.
    for that reason we should must remove insensitive "i" symbol from the end of the string pattern. with 'i' symbol and hybrid mode pattern ('O | C') preg_match function searched for any uppercase and lowercase letter and if find lowercase letter like 'o' in serialized text. this function return a wrong array that in result if statement ('//Prevent object injection or non arrays.') equal true then ot_decode function return '$fallback' value.

  1. Add menu select feature.
    Bringing all WordPress navigation menus to Option-Tree as a select box.

…' letter in the serialized string. for that reason we should must remove insensitive "i" symbol from the end of the string pattern. with 'i' symbol and hybrid mode pattern ('O | C') `preg_match` function searched for any upppercase and lowercase letter and if find lowercase letter like 'o' in serialized text. this function return a wrong array that in result `if` statement ('//Prevent object injection or non arrays.') equal true then `ot_decode` function return '$fallback' value.
@vheidari vheidari changed the title issues:710 - Hotfix on regex pattern and resolve problem to show list-item. issues:710 - Hotfix on regex pattern and resolve problem to show list-item. and add menu select feature. Oct 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant