luci-mod-admin-full: handle missing size for block devices
authorJo-Philipp Wich <jow@openwrt.org>
Fri, 17 Apr 2015 10:34:54 +0000 (12:34 +0200)
committerJo-Philipp Wich <jow@openwrt.org>
Fri, 17 Apr 2015 10:39:11 +0000 (12:39 +0200)
Signed-off-by: Jo-Philipp Wich <jow@openwrt.org>
modules/luci-mod-admin-full/luasrc/model/cbi/admin_system/fstab.lua
modules/luci-mod-admin-full/luasrc/model/cbi/admin_system/fstab/mount.lua

index 4c806dbc43da8f55de0d363aaf8e2be3f35e9724..fd6175835b3a4d6b69fbaa54368648a98ec2afee 100644 (file)
@@ -87,24 +87,30 @@ dev.cfgvalue = function(self, section)
 
        v = m.uci:get("fstab", section, "uuid")
        e = v and devices[v:lower()]
-       if v and e then
+       if v and e and e.size then
                return "UUID: %s (%s, %d MB)" %{ tp.pcdata(v), e.dev, e.size }
+       elseif v and e then
+               return "UUID: %s (%s)" %{ tp.pcdata(v), e.dev }
        elseif v then
                return "UUID: %s (<em>%s</em>)" %{ tp.pcdata(v), translate("not present") }
        end
 
        v = m.uci:get("fstab", section, "label")
        e = v and devices[v]
-       if v and e then
+       if v and e and e.size then
                return "Label: %s (%s, %d MB)" %{ tp.pcdata(v), e.dev, e.size }
+       elseif v and e then
+               return "Label: %s (%s)" %{ tp.pcdata(v), e.dev }
        elseif v then
                return "Label: %s (<em>%s</em>)" %{ tp.pcdata(v), translate("not present") }
        end
 
        v = Value.cfgvalue(self, section) or "?"
        e = v and devices[v]
-       if v and e then
+       if v and e and e.size then
                return "%s (%d MB)" %{ tp.pcdata(v), e.size }
+       elseif v and e then
+               return tp.pcdata(v)
        elseif v then
                return "%s (<em>%s</em>)" %{ tp.pcdata(v), translate("not present") }
        end
index 54beb63b42fe8fa7d7d2f5ee181ad7587a3f1ff6..2652e00f254ff84777fa832d83afca7321a9c2a6 100644 (file)
@@ -57,8 +57,10 @@ o = mount:taboption("general", Value, "uuid", translate("UUID"),
        translate("If specified, mount the device by its UUID instead of a fixed device node"))
 
 for i, d in ipairs(devices) do
-       if d.uuid then
+       if d.uuid and d.size then
                o:value(d.uuid, "%s (%s, %d MB)" %{ d.uuid, d.dev, d.size })
+       elseif d.uuid then
+               o:value(d.uuid, "%s (%s)" %{ d.uuid, d.dev })
        end
 end
 
@@ -71,8 +73,10 @@ o = mount:taboption("general", Value, "label", translate("Label"),
 o:depends("uuid", "")
 
 for i, d in ipairs(devices) do
-       if d.label then
+       if d.label and d.size then
                o:value(d.label, "%s (%s, %d MB)" %{ d.label, d.dev, d.size })
+       elseif d.label then
+               o:value(d.label, "%s (%s)" %{ d.label, d.dev })
        end
 end
 
@@ -85,7 +89,11 @@ o = mount:taboption("general", Value, "device", translate("Device"),
 o:depends({ uuid = "", label = "" })
 
 for i, d in ipairs(devices) do
-       o:value(d.dev, "%s (%d MB)" %{ d.dev, d.size })
+       if d.size then
+               o:value(d.dev, "%s (%d MB)" %{ d.dev, d.size })
+       else
+               o:value(d.dev)
+       end
 end