search

mtekk / Breadcrumb-NavXT

The longest maintained Breadcrumb plugin for WordPress
https://mtekk.us/code/breadcrumb-navxt
51 stars 25 forks source link

Block Widget Setting Saving #253

Closed mtekk closed 3 years ago

mtekk commented 3 years ago

It appears widget settings no longer work with the new WordPress block widget method.

mtekk commented 3 years ago

Looking into this a little, the only settings having issues are the checkboxes. Additionally, setting them to the correct value and pressing "update" does actually save the correct values. However, on page re-load the checkboxes end up getting inappropriately checked.

mtekk commented 3 years ago

Looking into this a little, the only settings having issues are the checkboxes. Additionally, setting them to the correct value and pressing "update" sort of saves the correct values. The issue appears that WordPress is improperly running the update method twice/effectively running it twice. As a result, a simple isset is not sufficient for checkboxes.