ios error when trying to add photo

[firebase]

ios error when trying to add photo

[ethanneff]

missing self.clientTable.reloadData() after self.messages.removeAll()

  override func viewDidAppear(animated: Bool) {
    self.messages.removeAll()
    self.clientTable.reloadData()
    // Listen for new messages in the Firebase database
    _refHandle = self.ref.child("messages").observeEventType(.ChildAdded, withBlock: { (snapshot) -> Void in
      self.messages.append(snapshot)
      let indexPath = NSIndexPath(forRow: self.messages.count-1, inSection: 0)
      self.clientTable.insertRowsAtIndexPaths([indexPath], withRowAnimation: .Automatic)
      self.clientTable.scrollToRowAtIndexPath(indexPath, atScrollPosition: .Bottom, animated: true)
    })
  }

[ethanneff]

@JosephGarcia by moving the code from viewDidAppear() to viewDidLoad(), you will remove the Realtime Database from the app whenever a user open the Camera Picker. The database listener is removed within viewWillDisappear().

  override func viewWillDisappear(animated: Bool) {
    self.ref.removeObserverWithHandle(_refHandle)
  }

[nicolasgarnier]

@iulukaya could you have a look?

[ulukaya]

It's related to your storage not enabled. If you just click the storage tab in your Firebase console, it'll set it up for you and upload will work next time you try.

[bdonkey]

@iulukaya
hmmm..not sure I understand this.
here is what i find in my storage tab:

and

I can upload a file from the storage tab in the console. However, in the code it bombs out to debugger both when I try to upload a file and when I cancel out of the upload attempt. Here is the code and dump when I try to upload an image:

FCViewController.Swift

variable dump

I don't want you or I to go down a rabbit hole here, so if there is anything else I can help you with to clarify, please let me know...thx...

scott

[bdonkey]

Update

Code bombs out in FCViewController function viewWillAppear

This Bombs

  override func viewWillAppear(animated: Bool) {
    self.messages.removeAll()
    // Listen for new messages in the Firebase database
    _refHandle = self.ref.child("messages").observeEventType(.ChildAdded, withBlock: { (snapshot) -> Void in
      self.messages.append(snapshot)
      self.clientTable.insertRowsAtIndexPaths([NSIndexPath(forRow: self.messages.count-1, inSection: 0)], withRowAnimation: .Automatic)
    })
  }

Commenting out self.messages.removeAll() the code does not bomb.
This runs but list prints out twice

  override func viewWillAppear(animated: Bool) {
//    self.messages.removeAll()
    // Listen for new messages in the Firebase database
    _refHandle = self.ref.child("messages").observeEventType(.ChildAdded, withBlock: { (snapshot) -> Void in
      self.messages.append(snapshot)
      self.clientTable.insertRowsAtIndexPaths([NSIndexPath(forRow: self.messages.count-1, inSection: 0)], withRowAnimation: .Automatic)
    })
  }

[JosephGarcia]

I have had the same problem. I moved the viewWillAppear code inside the viewDidLoad. Everything seems to be working fine. I am still a noobie but firebase is real time so I don't see why not just putting the observeEventType code into the viewDidLoad since its going to run still when you add a photo or text.

It is duplicating because viewDidAppear is running every time you dismiss the view controller associated with the Camera Picker. Just trying to help out. Hope this helps.

[ecaplain]

@ethanneff did you actually mean self.clientTable.reloadData() should be added to viewWillAppear() as instructed in the tutorial?

[ecaplain]

Slightly off topic, but I just noticed the image is saved wrapped in an optional. Looking for that and why "sent by: " is displayed in the chat view...

[bdonkey]

@JosephGarcia Thx for the comment. I was looking into what you were saying when @ethanneff helped clarify.

@ecaplain Yes is seems to me that @ethanneff meant viewWillAppear() and not viewDidLoad().

@ethanneff I tested this really quickly and you are right, the code stopped bombing into debugger. However it seems I am still getting double entries in my list when I add pictures. This could be a remnant of code on my part, so I will try look at further tomorrow...

here is my current code snippet where list prints twice:

override func viewWillAppear(animated: Bool) {
    self.messages.removeAll()
    self.clientTable.reloadData()
    // Listen for new messages in the Firebase database
    _refHandle = self.ref.child("messages").observeEventType(.ChildAdded, withBlock: { (snapshot) -> Void in
      self.messages.append(snapshot)
      self.clientTable.insertRowsAtIndexPaths([NSIndexPath(forRow: self.messages.count-1, inSection: 0)], withRowAnimation: .Automatic)
    })
  }

[ecaplain]

@bdonkey It's actually tableView() that gets called twice when an image is uploaded, not viewWillAppear, but I haven't figured out why yet...

[hasrants]

i am having same issue here. The original code from Firebase tutorial crashes app when selecting a photo. Reloading table as specified here in this issue thread, just creates a dual "sent by" with username and then photo when clicked.

[jmheist]

Same problem here. Loading the copleted app from firebase codelab, app crashes with "'Invalid update: invalid number of rows in section 0" As OP stated. Adding self.clientTable.reloadData() after self.messages.removeAll() in viewWillAppear() fixes the error, but loads the image twice in the table.

1. The message is only added to the firebase database once.
2. Deleting the entry from the firebase database does not auto remove it from the table list in app.

[erickuhn19]

Hey everyone,

This thread on stackoverflow shows you how to solve this issue.

Need to add this code to viewWillDisappear

self.ref.child("messages").removeObserverWithHandle(_refHandle)

[ulukaya]

Updated the code to fix this problem