Changes

course: remove the searchtype attributes instead of blanking them

Empty string is not neutral: BaseElasticSearcher skips a term when detail.getSearchType() is non-null and differs from the searcher type, so searchtype="" would silently drop filters. Absent -> null -> no type check.