Congegni
/
sintonia
7
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Revert "cc-4232: changed bit formatter to give empty string when bitrate is set to 0" 

This reverts commit 4e25a5ff442258c13f366a4557ca22181cdfac02.
This commit is contained in:
Rudi Grinberg 2012-08-15 15:05:35 -04:00
parent 7fe45a5bbc
commit a2dfcd6bc2
4 changed files with 7 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
airtime_mvc/application/models/formatters/BitrateFormatter.php
View File

@ -19,10 +19,6 @@ class BitrateFormatter {
{ {
$Kbps = bcdiv($this->_bitrate, 1000, 0); $Kbps = bcdiv($this->_bitrate, 1000, 0);
if($Kbps == 0) { return "{$Kbps} Kbps";
return "";
} else {
return "{$Kbps} Kbps";
}
} }
} }

2
python_apps/media-monitor2/media/monitor/metadata.py
View File

@ -51,7 +51,7 @@ airtime_special = {
"MDATA_KEY_DURATION" : "MDATA_KEY_DURATION" :
lambda m: format_length(getattr(m.info, u'length', 0.0)), lambda m: format_length(getattr(m.info, u'length', 0.0)),
"MDATA_KEY_BITRATE" : "MDATA_KEY_BITRATE" :
lambda m: getattr(m.info, "bitrate", ''), lambda m: getattr(m.info, "bitrate", 0),
"MDATA_KEY_SAMPLERATE" : "MDATA_KEY_SAMPLERATE" :
lambda m: getattr(m.info, u'sample_rate', 0), lambda m: getattr(m.info, u'sample_rate', 0),
"MDATA_KEY_MIME" : "MDATA_KEY_MIME" :

8
python_apps/media-monitor2/media/monitor/pure.py
View File

@ -209,12 +209,12 @@ def parse_int(s):
>>> parse_int("123saf") >>> parse_int("123saf")
123 123
>>> parse_int("asdf") >>> parse_int("asdf")
'' 0
""" """
if s.isdigit(): return s if s.isdigit(): return s
else: else:
try : return str(reduce(op.add, takewhile(lambda x: x.isdigit(), s))) try : return reduce(op.add, takewhile(lambda x: x.isdigit(), s))
except: return '' except: return 0
def normalized_metadata(md, original_path): def normalized_metadata(md, original_path):
""" """
@ -241,10 +241,10 @@ def normalized_metadata(md, original_path):
# could possibly lead to subtle bugs down the road. Plus the following # could possibly lead to subtle bugs down the road. Plus the following
# approach gives us the flexibility to use different defaults for different # approach gives us the flexibility to use different defaults for different
# attributes # attributes
new_md = remove_whitespace(new_md)
new_md = apply_rules_dict(new_md, format_rules) new_md = apply_rules_dict(new_md, format_rules)
new_md = default_to(dictionary=new_md, keys=['MDATA_KEY_TITLE'], new_md = default_to(dictionary=new_md, keys=['MDATA_KEY_TITLE'],
default=no_extension_basename(original_path)) default=no_extension_basename(original_path))
new_md = remove_whitespace(new_md)
new_md = default_to(dictionary=new_md, keys=path_md, new_md = default_to(dictionary=new_md, keys=path_md,
default=u'') default=u'')
new_md = default_to(dictionary=new_md, keys=['MDATA_KEY_FTYPE'], new_md = default_to(dictionary=new_md, keys=['MDATA_KEY_FTYPE'],

5
python_apps/media-monitor2/tests/test_pure.py
View File

@ -67,9 +67,4 @@ class TestMMP(unittest.TestCase):
self.assertTrue( mmp.sub_path(d1, f1) ) self.assertTrue( mmp.sub_path(d1, f1) )
self.assertTrue( mmp.sub_path(d2, f1) ) self.assertTrue( mmp.sub_path(d2, f1) )
def test_parse_int(self):
self.assertEqual( mmp.parse_int("123"), "123" )
self.assertEqual( mmp.parse_int("123asf"), "123" )
self.assertEqual( mmp.parse_int("asdf"), "" )
if __name__ == '__main__': unittest.main() if __name__ == '__main__': unittest.main()