[Lua][Solved] fs.open() issue

function checkMail()
local option = nil
local file = nil
local temp = nil
local emailName = ""
local startIndex = nil
local endIndex = nil
local filepath = "Emails/"

if fs.exists("Emails") == false then
  fs.makeDir("Emails")
end

rednet.broadcast("readmail")
a, messagep = rednet.receive(1)
if messagep ~= nil then
  startIndex = messagep:find("Subject: ") + 9
  endIndex = messagep:find("Message:") - 1

  emailName = string.sub(messagep,startIndex,endIndex)
  print(emailName)

  filepath = filepath..emailName
  print(filepath)
  sleep(1)

  file = fs.open(filepath, "w") -- offending code
							  -- I have already checked that the filepath is valid using print()
							  -- above, so I know that is not the issue. It just seems that the
							  -- method is not working...
  if file then
   print("test")
   sleep(1)
   file.write(messagep)
   file.close()
  end
end

if fs.isDir("Emails") then
  emails = fs.list("Emails")
  emails[#emails + 1] = "Exit"
end

option = CUI(emails)

if option ~= #emails then
  file = fs.open("Emails/" ..emails[option], "r") -- Strangely enough, it works down here.
  temp = file.readAll()
  file:close()
  print(temp.. "\n")
  write("Press Enter to Continue...")
  read()
  os.run({}, "emailClient")
else
  emailMenu()
end
end

if fs.isDir(filepath) then
print("Can't open "..filepath..", file is a directory!")
else
file = fs.open(filepath, "w")
  if file then
   print("test")
   sleep(1)
   file.write(messagep)
   file.close()
end
end

thesbros, on 30 June 2012 - 05:17 AM said:

Try:

if not fs.isDir(filepath) then
print("Can't open "..filepath..", file is a directory!")
else
file = fs.open(filepath, "w")
  if file then
   print("test")
   sleep(1)
   file.write(messagep)
   file.close()
end
end

And if it says it's a directory, well, then it's a directory and it shouldn't be.
P.S: The same thing has happened to me before, if this doesn't work then): (oddly enough this worked for me) try reinstalling CC

if fs.isDir(filepath) then

if not fs.isDir(filepath)

file = fs.open("Emails/test")
file:close()

-- This is my ComputerCraft eMail client code.
local args = {...}
local menu1 = {"Check Email","Send Email","Delete Email","Exit"}
local emails = nil
function CUI(m)
n=1
l=#m
while true do
  term.clear()
  term.setCursorPos(1,2)
  for i=1, l, 1 do
   if i==n then print(i, " ["..m[i].."]") else print(i, " ", m[i]) end
  end
  print("Select a number[arrow up/arrow down]")
  a, b= os.pullEventRaw()
  if a == "key" then
   if b==200 and n>1 then
	n=n-1
   end
   if b==208 and n<l then
	n=n+1
   end
   if b==28 then
	break
   end
  end
end
term.clear() term.setCursorPos(1,1)
return n
end
function emailMenu()
local option = nil
option = CUI(menu1)
if option == 1 then
  checkMail()
end
if option == 2 then
  sendMail()
end
if option == 4 then
  os.run({},"lemonOS", true)
  os.exit()
end
end
function checkMail()
local option = nil
local file = nil
local temp = nil
local emailName = ""
local startIndex = nil
local endIndex = nil
local filepath = "Emails/"

if fs.exists("Emails") == false then
  fs.makeDir("Emails")
end

rednet.broadcast("readmail")
a, messagep = rednet.receive(1)
if messagep ~= nil then
  startIndex = messagep:find("Subject: ") + 9
  endIndex = messagep:find("Message:") - 1

  emailName = string.sub(messagep,startIndex,endIndex)
  print(emailName)

  filepath = filepath..emailName
  print(filepath)
  sleep(1)


  if not fs.isDir(filepath) then
   print("Can't open "..filepath..", file is a directory!")
  else
   file = fs.open(filepath, "w")
	 if file then
	  print("test")
	  sleep(1)
	  file.write(messagep)
	  file.close()
   end
  end
end

if fs.isDir("Emails") then
  emails = fs.list("Emails")
  emails[#emails + 1] = "Exit"
end

option = CUI(emails)

if option ~= #emails then
  file = fs.open("Emails/" ..emails[option], "r")
  temp = file.readAll()
  file:close()
  print(temp.. "n")
  write("Press Enter to Continue...")
  read()
  os.run({}, "emailClient")
else
  emailMenu()
end
end
function sendMail()
write("Do you want to send mail? (Y/N) ")
input = read()
input = string.upper(input)
if input == "Y" then
  rednet.broadcast("sendmail")
  write("To: (in ID) ")
  input = read()
  rednet.broadcast(input)
  write("Subject: ")
  input = read()
  rednet.broadcast(input)
  print("Message body:")
  input = read()
  rednet.broadcast(input)
end
emailMenu()
end
emailMenu()

filepath = filepath..emailName

filepath = fs.combine(filepath, emailName)

if not fs.isDir(filepath) then

MysticT, on 01 July 2012 - 11:00 PM said:

Well, the not shouldn't be in this line:

if not fs.isDir(filepath) then

since you want to check if it is a directory. Other than that, I see no error.

if fs.exists("Emails") == false then
  fs.makeDir("Emails")
end

local file = fs.open("test123", "w")
if file then
  file.close()
  print("The file was created.")
  fs.delete("test123")
else
  print("Error creating file.")
end

UJustGotTrolled, on 01 July 2012 - 11:08 PM said:

MysticT, on 01 July 2012 - 11:00 PM said:

Well, the not shouldn't be in this line:

if not fs.isDir(filepath) then

since you want to check if it is a directory. Other than that, I see no error.

Ah, didn't catch that. That wasn't my code anyway. Thesbros suggested it. As such, it can't be causing the issue.

MysticT, on 02 July 2012 - 01:16 AM said:

In ther terminal, type:
> cd /Emails
> ls
And post what you see.