Thierry says this is not used out in the wild, and I can't find
any reference to say that it must be there, so remove it.
if (_release_territory) {
auto territory = _release_territory->subtag();
isdcf_name += "_" + to_upper (territory);
- if (_ratings.empty ()) {
- isdcf_name += "-NR";
- } else {
+ if (!_ratings.empty()) {
auto label = _ratings[0].label;
boost::erase_all(label, "+");
boost::erase_all(label, "-");
/* Check that the proper codes are used, not just part of the language code; in this case, QBP instead of PT (#2235) */
film->set_audio_language(dcp::LanguageTag("pt-BR"));
BOOST_CHECK_EQUAL (film->isdcf_name(false), "LikeShouting_XSN-2_F-133_QBP-fr_US-R_71-HI-VI_4K_DI_20140704_PPF_SMPTE_OV");
+
+ /* Check that nothing is added for non-existant ratings */
+ film->set_ratings({});
+ BOOST_CHECK_EQUAL (film->isdcf_name(false), "LikeShouting_XSN-2_F-133_QBP-fr_US_71-HI-VI_4K_DI_20140704_PPF_SMPTE_OV");
+
}