Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

attachments hash (Josh) #403

Closed
wants to merge 4 commits into from
Closed

attachments hash (Josh) #403

wants to merge 4 commits into from

Conversation

norv
Copy link
Contributor

@norv norv commented May 14, 2013

Picked and rebased the attachments part of #397. Open for review.

get rid of the $new parameter in getAttachmentFilename()
Signed-off-by:Joshua Dickerson <joshua.a.dickerson@gmail.com>
Signed-off-by:Joshua Dickerson <joshua.a.dickerson@gmail.com>
…of this query. If $file_hash is empty, it is because it isn't set in the database.

Signed-off-by:Joshua Dickerson <joshua.a.dickerson@gmail.com>
Signed-off-by:Joshua Dickerson <joshua.a.dickerson@gmail.com>
Spuds added a commit to Spuds/Elkarte that referenced this pull request Nov 2, 2013
… Subs.php

! use the filehash in the call, not the legacy call?
! parts of elkarte#403

Signed-off-by: Spuds <spuds@spudsdesign.com>
Spuds pushed a commit to Spuds/Elkarte that referenced this pull request Nov 2, 2013
! part of ref : elkarte#403

Signed-off-by: Spuds <spuds@spudsdesign.com>
@Spuds Spuds mentioned this pull request Nov 2, 2013
@Spuds
Copy link
Contributor

Spuds commented Nov 6, 2013

As much of this as we can do has been done ... so I'm closing this one

@Spuds Spuds closed this Nov 6, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants