Hotfix 1.3.1

[public.git] / lua / publish.lua

diff --git a/lua/publish.lua b/lua/publish.lua
index b98e8f4a344daea49f14e5f8f8df9d7672e12e16..f5620183cfdd906030a0bba48d321c6d40f654f0 100644 (file)
--- a/lua/publish.lua
+++ b/lua/publish.lua
@@ -1,7 +1,7 @@
 require "apache2"
 
 function publish(dir, pattern)
 require "apache2"
 
 function publish(dir, pattern)

-    local cmd = ("%s/publish.sh %s"):format(dir, pattern)
+    local cmd = ("%s/publish.sh '%s'"):format(dir, pattern)

     local handle = io.popen(cmd)
     local output = handle:read('*a')
     handle:close()
     local handle = io.popen(cmd)
     local output = handle:read('*a')
     handle:close()


@@ -21,18 +21,26 @@ function handle(r)
     r.content_type = "text/html"
 
     local args = r:parseargs()
     r.content_type = "text/html"
 
     local args = r:parseargs()

-    local pattern = args.pattern 
+    local pattern = args.pattern

 
     if pattern then
 
     if pattern then

-        local nonce = publish(r.context_document_root, pattern)
-
-        -- if FQDN override is set,
-        -- use override instead of my own server name and port number
-        local fqdn = os.getenv('PUBLIC_FQDN_OVERRIDE')
-        fqdn = fqdn or (r.is_https and "https" or "http") .. "://" .. r.server_name .. ":" ..r.port
+        -- filename might have spaces that apache replaces with '+'
+        pattern = string.gsub(pattern, '+', ' ')

 
 

-        local uri = ("%s/%s/"):format(fqdn, nonce)
-        r:puts( ([[<a href="%s">%s</a>]]):format(uri, uri) )
+        local nonce = publish(r.context_document_root, pattern)
+        if nonce == "" then
+            r:puts( "No links created" )
+        else
+            -- if FQDN override is set,
+            -- use override instead of my own server name and port number
+            local fqdn = os.getenv('PUBLIC_FQDN_OVERRIDE')
+            fqdn = fqdn or
+                (r.is_https and "https" or "http") ..
+                "://" .. r.server_name .. ":" ..r.port
+
+            local uri = ("%s/%s/"):format(fqdn, nonce)
+            r:puts( ([[<a href="%s">%s</a>]]):format(uri, uri) )
+        end

     else
         local template = get_form(r.context_document_root)
         r:puts( template )
     else
         local template = get_form(r.context_document_root)
         r:puts( template )