erictansh

I think the problem lies with the filter. The filter is not working properly. I cannot filter the unoptimized images, and need to click through thousands of images to find them.

Yes changing to WP_REDIS_PLUGIN_DIR fix the fatal error

Tried but still didn’t work

Note: there are more than 1 slug.

add_filter('site-reviews/create/review-values', function ($values) {
  
  // Find the treatment categories of the posted review

  $terms = get_the_terms( $values['post_id'], 'treatment_cat' );

  $assigned_terms = wp_list_pluck($terms, 'slug');
  
  // Assign the terms to the review
  $values['assigned_terms'] = $assigned_terms;

  return $values;
});

Great

Does $values[‘assigned_terms’] accept slugs? I tried yesterday and didn’t work.

That’s why I need to convert to $term->id instead.

Thanks for the quick fix!

Just tested it and is working!

Just one suggestion – any possibility of using the CPT’s taxonomy to filter the reviews instead of using site-review-category.

Currently I have to write a hook to sync up the CPT’s terms and Site Reviews’ terms. This workaround will only works if the both categories’ slug matched exactly.

It would be great if we can use the CPT’s taxonomy so to save the trouble having to match and sync both categories.

add_filter('site-reviews/create/review-values', function ($values) {
  
  
  // Find the treatment categories of the posted review

  $terms = get_the_terms( $values['post_id'], 'treatment_cat' );

  $slugs = wp_list_pluck($terms, 'slug');

  // Match the treatment categories to that of the review categories using the slug
  foreach ($slugs as $slug) {
    $term = get_term_by( 'slug', $slug , 'site-review-category' );
    $assigned_terms[] = $term->term_id;
  }
  
  // Assign the terms to the review
  $values['assigned_terms'] = $assigned_terms;

  return $values;
});

One more thing – you forget to add the “break” in the recent patch

missing s for %s

and

Contact List to Group List

• This reply was modified 5 years ago by erictansh.

Spotted a minor bug

‘tooltip’ => sprintf(‘
<h6>%s</h6>%s’,
esc_html__( ‘Group List’, ‘connector-gravityforms-mailerlite’ ),
esc_html__( ‘Select the Mailer Lite list you would like to add your contacts to.’, ‘connector-gravityforms-mailerlite’ )
),

• This reply was modified 5 years ago by erictansh.

without the break, it will loop through the entire array. And if the last item of the array is not the matching group list, it will replace the $group_name with “None”.

https://pastie.org/10982404#141

https://pastie.org/10981736#1,108

https://pastie.org/10981739#17,39

https://pastie.org/10980406#105,107-109

https://pastie.org/10979572#138,140-143

Updated for Gravity Forms version 2.0.7

https://pastie.org/10961496#13-19,38-41

Thanks Frank – work like a charm. Will be checking out your critical css addon soon ??