@teja-amil - the param 'show_category_heading_title' in intended to show/hide the category title, not the "Subcategories" text.

I believe instead, the "if" statement should be revised to also check whether we want to hide empty child categories.

As it stands now, you can hide the Subcategories text only if you set "Subcategory Levels" (maxLevel) to None. You should be able to leave levels to something other than None while still having it obey "Empty Categories" set to hide (not necessary to show "Subcategories" text if all the children are empty).

Hope that makes sense. If you update the PR, I'll provide a test. :)

@radiant-tech Thanks for the heads-up! Updated the PR. Please go ahead and fix this.

@teja-amil - Sorry, but the fix I provided was not correct, since it's only looking at the "empty cats" param setting instead of whether we have at least one subcat with an item to show. I've proposed a better solution at #3175 -- please feel free to test and report back there or in it's related Tracker.

While waiting for the fix to be accepted, you could avoid a core hack and override the layout content category file in your template. Where I am using $displayData->get('category')->childrennumitems, you would need to use ($displayData->get('category')->getNumItems(true) - $displayData->get('category')->getNumItems(false)) > 0

I've proposed a better solution at #3175

Can this one be closed then?

Yes, I would recommend closing this one.

Closing as requested.