pods icon indicating copy to clipboard operation
pods copied to clipboard

Error Editing existing pods(other language using WPML)

Open jlapitan opened this issue 2 years ago • 6 comments

Description

Was using an older version of Pods(2.7.31), updated to version 2.8.3 and was trying to add edit/add new content on another language when I encountered the issue.

When I tried to edit an existing post on another language I got this error:

PHP message: PHP Fatal error: Uncaught Error: Call to a member function get_field() on bool in /www/xxxxx_896/public/wp-content/plugins/pods/classes/PodsMeta.php:3523

All other details from wp support forum

Issue might be related to introducing of new set of fields?

Version

2.8.3

Testing Instructions

  1. Install Pods(v2.7.31) and WPML
  2. Create new post type and at least two fields
  3. Set post type to be translatable.
  4. Add one post on each of the language(e.g. english and arabic)
  5. Upgrade to latest version of Pods(2.8.3)
  6. Edit secondary language.(you will get the error here)

Screenshots / Screencast

Before the upgrade:

s1: scrnli_11_14_2021_11-53-53 AM s2: scrnli_11_14_2021_11-55-07 AM s3: scrnli_11_14_2021_11-55-32 AM

s4(i can edit arabic content): scrnli_11_14_2021_11-56-02 AM

Update to v2.8.3

s5(updated plugin) scrnli_11_14_2021_11-58-19 AM s6(encountered the problem) scrnli_11_14_2021_12-02-11 PM

Possible Workaround

Rollback to lower version(2.7.31)

Site Health Information

No response

Pods Package

No response

jlapitan avatar Nov 14 '21 08:11 jlapitan

@sc0ttkclark for the Pods package, which one is needed? the one on v2.7.31 or on 2.8.3?

jlapitan avatar Nov 14 '21 08:11 jlapitan

{
   "meta":{
      "version":"2.7.31",
      "build":1636877343
   },
   "pods":{
      "2249":{
         "id":2249,
         "name":"external_news",
         "label":"External News",
         "description":"",
         "type":"post_type",
         "storage":"meta",
         "object":"",
         "alias":"",
         "fields":{
            "news_source":{
               "id":2250,
               "name":"news_source",
               "label":"News Source",
               "description":"E.g. Reuters, CNN, BBC",
               "help":"",
               "class":"",
               "type":"text",
               "weight":0,
               "pick_object":"",
               "pick_val":"",
               "sister_id":"",
               "required":"1",
               "text_allow_shortcode":"0",
               "text_allow_html":"0",
               "text_allowed_html_tags":"strong em a ul ol li b i",
               "text_max_length":"255",
               "pick_post_status":[
                  "publish"
               ],
               "admin_only":"0",
               "restrict_role":"0",
               "restrict_capability":"0",
               "hidden":"0",
               "read_only":"0",
               "roles_allowed":[
                  "administrator"
               ],
               "rest_read":"0",
               "rest_write":"0",
               "rest_pick_response":"array",
               "rest_pick_depth":"2",
               "unique":"0",
               "text_repeatable":"0",
               "text_placeholder":""
            },
            "news_source_url":{
               "id":2251,
               "name":"news_source_url",
               "label":"News Source URL",
               "description":"",
               "help":"",
               "class":"",
               "type":"website",
               "weight":1,
               "pick_object":"",
               "pick_val":"",
               "sister_id":"",
               "required":"1",
               "website_format":"normal",
               "website_allow_port":"0",
               "website_clickable":"0",
               "website_new_window":"0",
               "website_max_length":"255",
               "website_html5":"0",
               "pick_post_status":[
                  "publish"
               ],
               "admin_only":"0",
               "restrict_role":"0",
               "restrict_capability":"0",
               "hidden":"0",
               "read_only":"0",
               "roles_allowed":[
                  "administrator"
               ],
               "rest_read":"0",
               "rest_write":"0",
               "rest_pick_response":"array",
               "rest_pick_depth":"2",
               "unique":"0",
               "website_repeatable":"0",
               "website_placeholder":""
            }
         },
         "show_in_menu":"1",
         "label_singular":"External News",
         "public":"1",
         "show_ui":"1",
         "supports_title":"1",
         "supports_editor":"0",
         "publicly_queryable":"1",
         "exclude_from_search":"1",
         "capability_type":"post",
         "capability_type_custom":"external_news",
         "capability_type_extra":"1",
         "has_archive":"0",
         "hierarchical":"0",
         "rewrite":"1",
         "rewrite_with_front":"1",
         "rewrite_feeds":"0",
         "rewrite_pages":"1",
         "query_var":"1",
         "can_export":"1",
         "default_status":"draft",
         "supports_author":"0",
         "supports_thumbnail":"1",
         "supports_excerpt":"0",
         "supports_trackbacks":"0",
         "supports_custom_fields":"0",
         "supports_comments":"0",
         "supports_revisions":"0",
         "supports_page_attributes":"0",
         "supports_post_formats":"0",
         "built_in_taxonomies_category":"0",
         "built_in_taxonomies_element_category":"0",
         "built_in_taxonomies_faq_category":"0",
         "built_in_taxonomies_fusion_tb_category":"0",
         "built_in_taxonomies_link_category":"0",
         "built_in_taxonomies_portfolio_category":"0",
         "built_in_taxonomies_portfolio_skills":"0",
         "built_in_taxonomies_portfolio_tags":"0",
         "built_in_taxonomies_post_tag":"0",
         "built_in_taxonomies_slide-page":"0",
         "built_in_taxonomies_template_category":"0",
         "menu_position":"0",
         "show_in_nav_menus":"1",
         "show_in_admin_bar":"1",
         "pfat_enable":"0",
         "pfat_append_single":"append",
         "pfat_filter_single":"the_content",
         "pfat_append_archive":"append",
         "pfat_filter_archive":"the_excerpt",
         "pfat_run_outside_loop":"0",
         "rest_enable":"0",
         "read_all":"0",
         "write_all":"0",
         "menu_icon":"dashicons-admin-site-alt3",
         "built_in_taxonomies_external_news_tag":"1"
      }
   }
}

jlapitan avatar Nov 14 '21 08:11 jlapitan

{
    "meta": {
        "version": "2.8.3",
        "build": 1636877508
    },
    "pods": [
        {
            "name": "external_news",
            "id": 2249,
            "label": "External News",
            "description": "",
            "label_singular": "External News",
            "public": "1",
            "show_ui": "1",
            "supports_title": "1",
            "supports_editor": "0",
            "type": "post_type",
            "storage": "meta",
            "publicly_queryable": "1",
            "exclude_from_search": "1",
            "capability_type": "post",
            "capability_type_custom": "external_news",
            "capability_type_extra": "1",
            "has_archive": "0",
            "hierarchical": "0",
            "rewrite": "1",
            "rewrite_with_front": "1",
            "rewrite_feeds": "0",
            "rewrite_pages": "1",
            "query_var": "1",
            "can_export": "1",
            "default_status": "draft",
            "supports_author": "0",
            "supports_thumbnail": "1",
            "supports_excerpt": "0",
            "supports_trackbacks": "0",
            "supports_custom_fields": "0",
            "supports_comments": "0",
            "supports_revisions": "0",
            "supports_page_attributes": "0",
            "supports_post_formats": "0",
            "built_in_taxonomies_category": "0",
            "built_in_taxonomies_element_category": "0",
            "built_in_taxonomies_faq_category": "0",
            "built_in_taxonomies_fusion_tb_category": "0",
            "built_in_taxonomies_link_category": "0",
            "built_in_taxonomies_portfolio_category": "0",
            "built_in_taxonomies_portfolio_skills": "0",
            "built_in_taxonomies_portfolio_tags": "0",
            "built_in_taxonomies_post_tag": "0",
            "built_in_taxonomies_slide-page": "0",
            "built_in_taxonomies_template_category": "0",
            "show_in_menu": "1",
            "menu_position": "0",
            "show_in_nav_menus": "1",
            "show_in_admin_bar": "1",
            "pfat_enable": "0",
            "pfat_append_single": "append",
            "pfat_filter_single": "the_content",
            "pfat_append_archive": "append",
            "pfat_filter_archive": "the_excerpt",
            "pfat_run_outside_loop": "0",
            "rest_enable": "0",
            "read_all": "0",
            "write_all": "0",
            "menu_icon": "dashicons-admin-site-alt3",
            "built_in_taxonomies_external_news_tag": "1",
            "groups": [
                {
                    "name": "more_fields",
                    "label": "More Fields",
                    "fields": [
                        {
                            "name": "news_source",
                            "id": 2250,
                            "label": "News Source",
                            "description": "E.g. Reuters, CNN, BBC",
                            "weight": 0,
                            "type": "text",
                            "required": "1",
                            "text_allow_shortcode": "0",
                            "text_allow_html": "0",
                            "text_allowed_html_tags": "strong em a ul ol li b i",
                            "text_max_length": "255",
                            "phone_enable_phone_extension": "1",
                            "paragraph_allow_html": "1",
                            "paragraph_oembed": "0",
                            "paragraph_wptexturize": "1",
                            "paragraph_convert_chars": "1",
                            "paragraph_wpautop": "1",
                            "paragraph_allow_shortcode": "0",
                            "wysiwyg_media_buttons": "1",
                            "wysiwyg_oembed": "0",
                            "wysiwyg_wptexturize": "1",
                            "wysiwyg_convert_chars": "1",
                            "wysiwyg_wpautop": "1",
                            "wysiwyg_allow_shortcode": "0",
                            "code_allow_shortcode": "0",
                            "oembed_enabled_providers_amazoncn": "0",
                            "oembed_enabled_providers_amazoncouk": "0",
                            "oembed_enabled_providers_amazoncom": "0",
                            "oembed_enabled_providers_amazoncomau": "0",
                            "oembed_enabled_providers_amazonin": "0",
                            "oembed_enabled_providers_animotocom": "0",
                            "oembed_enabled_providers_cloudupcom": "0",
                            "oembed_enabled_providers_crowdsignalcom": "0",
                            "oembed_enabled_providers_dailymotioncom": "0",
                            "oembed_enabled_providers_flickrcom": "0",
                            "oembed_enabled_providers_imgurcom": "0",
                            "oembed_enabled_providers_issuucom": "0",
                            "oembed_enabled_providers_kickstartercom": "0",
                            "oembed_enabled_providers_meetupcom": "0",
                            "oembed_enabled_providers_mixcloudcom": "0",
                            "oembed_enabled_providers_redditcom": "0",
                            "oembed_enabled_providers_reverbnationcom": "0",
                            "oembed_enabled_providers_screencastcom": "0",
                            "oembed_enabled_providers_scribdcom": "0",
                            "oembed_enabled_providers_slidesharenet": "0",
                            "oembed_enabled_providers_smugmugcom": "0",
                            "oembed_enabled_providers_someecardscom": "0",
                            "oembed_enabled_providers_soundcloudcom": "0",
                            "oembed_enabled_providers_speakerdeckcom": "0",
                            "oembed_enabled_providers_spotifycom": "0",
                            "oembed_enabled_providers_tedcom": "0",
                            "oembed_enabled_providers_tiktokcom": "0",
                            "oembed_enabled_providers_tumblrcom": "0",
                            "oembed_enabled_providers_twittercom": "0",
                            "oembed_enabled_providers_vimeocom": "0",
                            "oembed_enabled_providers_wordpresscom": "0",
                            "oembed_enabled_providers_wordpresstv": "0",
                            "oembed_enabled_providers_youtubecom": "0",
                            "pick_post_status": [
                                "publish"
                            ],
                            "admin_only": "0",
                            "restrict_role": "0",
                            "restrict_capability": "0",
                            "hidden": "0",
                            "read_only": "0",
                            "roles_allowed": [
                                "administrator"
                            ],
                            "rest_read": "0",
                            "rest_write": "0",
                            "rest_pick_response": "array",
                            "rest_pick_depth": "2",
                            "unique": "0",
                            "text_repeatable": "0"
                        },
                        {
                            "name": "news_source_url",
                            "id": 2251,
                            "label": "News Source URL",
                            "description": "",
                            "weight": 1,
                            "type": "website",
                            "required": "1",
                            "text_allow_shortcode": "0",
                            "text_allow_html": "0",
                            "website_format": "normal",
                            "website_allow_port": "0",
                            "website_clickable": "0",
                            "website_new_window": "0",
                            "website_max_length": "255",
                            "website_html5": "0",
                            "phone_enable_phone_extension": "1",
                            "paragraph_allow_html": "1",
                            "paragraph_oembed": "0",
                            "paragraph_wptexturize": "1",
                            "paragraph_convert_chars": "1",
                            "paragraph_wpautop": "1",
                            "paragraph_allow_shortcode": "0",
                            "wysiwyg_media_buttons": "1",
                            "wysiwyg_oembed": "0",
                            "wysiwyg_wptexturize": "1",
                            "wysiwyg_convert_chars": "1",
                            "wysiwyg_wpautop": "1",
                            "wysiwyg_allow_shortcode": "0",
                            "code_allow_shortcode": "0",
                            "oembed_enabled_providers_amazoncn": "0",
                            "oembed_enabled_providers_amazoncouk": "0",
                            "oembed_enabled_providers_amazoncom": "0",
                            "oembed_enabled_providers_amazoncomau": "0",
                            "oembed_enabled_providers_amazonin": "0",
                            "oembed_enabled_providers_animotocom": "0",
                            "oembed_enabled_providers_cloudupcom": "0",
                            "oembed_enabled_providers_crowdsignalcom": "0",
                            "oembed_enabled_providers_dailymotioncom": "0",
                            "oembed_enabled_providers_flickrcom": "0",
                            "oembed_enabled_providers_imgurcom": "0",
                            "oembed_enabled_providers_issuucom": "0",
                            "oembed_enabled_providers_kickstartercom": "0",
                            "oembed_enabled_providers_meetupcom": "0",
                            "oembed_enabled_providers_mixcloudcom": "0",
                            "oembed_enabled_providers_redditcom": "0",
                            "oembed_enabled_providers_reverbnationcom": "0",
                            "oembed_enabled_providers_screencastcom": "0",
                            "oembed_enabled_providers_scribdcom": "0",
                            "oembed_enabled_providers_slidesharenet": "0",
                            "oembed_enabled_providers_smugmugcom": "0",
                            "oembed_enabled_providers_someecardscom": "0",
                            "oembed_enabled_providers_soundcloudcom": "0",
                            "oembed_enabled_providers_speakerdeckcom": "0",
                            "oembed_enabled_providers_spotifycom": "0",
                            "oembed_enabled_providers_tedcom": "0",
                            "oembed_enabled_providers_tiktokcom": "0",
                            "oembed_enabled_providers_tumblrcom": "0",
                            "oembed_enabled_providers_twittercom": "0",
                            "oembed_enabled_providers_vimeocom": "0",
                            "oembed_enabled_providers_wordpresscom": "0",
                            "oembed_enabled_providers_wordpresstv": "0",
                            "oembed_enabled_providers_youtubecom": "0",
                            "pick_post_status": [
                                "publish"
                            ],
                            "admin_only": "0",
                            "restrict_role": "0",
                            "restrict_capability": "0",
                            "hidden": "0",
                            "read_only": "0",
                            "roles_allowed": [
                                "administrator"
                            ],
                            "rest_read": "0",
                            "rest_write": "0",
                            "rest_pick_response": "array",
                            "rest_pick_depth": "2",
                            "unique": "0",
                            "website_repeatable": "0"
                        }
                    ]
                }
            ]
        }
    ]
}

jlapitan avatar Nov 14 '21 08:11 jlapitan

This is resolved in Pods 2.8.4 already but I'm not sure what is causing the pod objects to not be found here. Maybe it's getting confused somewhere else but the error no longer happens. I'm going to keep this open to look at in 2.8.5 in case there's any other logic up in the pod set up to sort out.

Also I'd like to add other meta keys to bypass anyways including these.

sc0ttkclark avatar Nov 14 '21 22:11 sc0ttkclark

@sc0ttkclark how can i test the v2.8.4? would like to test it on my staging site, thanks!

jlapitan avatar Nov 15 '21 05:11 jlapitan

https://wordpress.org/support/topic/error-editing-existing-podsother-language/#post-15063463

@sc0ttkclark i have updated to the latest version, no more error thrown, but the fields are missing when editing PODs on another language

JoryHogeveen avatar Nov 17 '21 09:11 JoryHogeveen