Skip to content

Commit 170980f

Browse files
committed
- phpcs
- remove WordPress 5.9 from testing matrix
1 parent e109047 commit 170980f

File tree

2 files changed

+4
-6
lines changed

2 files changed

+4
-6
lines changed

.github/workflows/testing-integration.yml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -33,12 +33,8 @@ jobs:
3333
acf_pro: true
3434
wpgraphql_content_blocks: true
3535
coverage: 1
36-
- php: '8.1'
37-
wordpress: '5.9'
3836
- php: '7.4'
3937
wordpress: '6.1'
40-
- php: '7.3'
41-
wordpress: '5.9'
4238
fail-fast: false
4339
name: WordPress ${{ matrix.wordpress }}, PHP ${{ matrix.php }}, ACF ${{matrix.acf_version}}, ACF & ACF EXTENDED PRO ${{matrix.acf_pro}}, WPGRAPHQL Content Blocks ${{matrix.wpgraphql_content_blocks}}
4440
steps:

src/AcfGraphQLFieldType.php

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -166,9 +166,11 @@ public function get_admin_field_settings( array $field, Settings $settings ) {
166166
// Get the admin fields for the field type
167167
$admin_fields = $this->get_admin_fields( $field, $default_admin_settings, $settings );
168168

169+
$excluded_admin_fields = $this->get_config( 'exclude_admin_fields' );
170+
169171
// Remove excluded fields
170-
if ( isset( $this->config['exclude_admin_fields'] ) && is_array( $this->config['exclude_admin_fields'] ) ) {
171-
foreach ( $this->config['exclude_admin_fields'] as $excluded ) {
172+
if ( is_array( $excluded_admin_fields ) ) {
173+
foreach ( $excluded_admin_fields as $excluded ) {
172174
unset( $admin_fields[ $excluded ] );
173175
}
174176
}

0 commit comments

Comments
 (0)