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.

Sign up for GitHub

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

Jump to bottom

Bug fixes for swift integration #27

Closed
wants to merge 26 commits into from
Closed

Bug fixes for swift integration #27

wants to merge 26 commits into from

Conversation

benliles
Copy link
Contributor

Various bug fixes for swift external file support.

  • Space free reported as 0 so uploads were canceled
  • fopen would not create files, uploads failed
  • stat assumed path was an object, no way to stat directories for filecache scanner
  • unlink, called when deleting from web ui, assumed path was an object, would not remove directories

benliles and others added 26 commits October 9, 2012 15:07
@benliles
Clean up directories when they are '.'
efd1004
@benliles
More cleanup for period path names
a02661a
@benliles
Return null when trying to get the root container as an object
0a38b28
@benliles
when creating an object, create the container that would hold it if i…
076d4ec 
…t doesn't exist instead, when checking to see if root container exists, check / instead of ->root because ->root is auto prepended
@benliles
Moved checkout for an object in fopen to where it returns a stream fo…
17fa001 
…r it when reading to remove null return when object doesn't exist
@benliles
Added a large 'free space' return value since 0 restricts uploading a…
c147470 
…nything in the interface, not an easy way to get free space with swift
@benliles
Added container specific stat code
36a2490
@benliles
Merge owncloud/core master
9cd42ca
@benliles
Folder removing through unlink now calls rmdir instead of failing
af9108a
@benliles
Merge remote-tracking branch 'upstream/master'
3d870f9
@bartv2
Use __DIR__ instead of __FILE__ to get SERVERROOT
fe40277
@bartv2
No need to use array_search
c5983d3
@bartv2
Remove unused search code
d4feb77
@bartv2
Add logging when stripping apps from autoload include path
2c3674e
@DeepDiver1975
move autotest to phpunit
39e1c22
@benliles
Clean up directories when they are '.'
97881f8
@benliles
More cleanup for period path names
d70a383
@benliles
Return null when trying to get the root container as an object
0574c97
@benliles
when creating an object, create the container that would hold it if i…
dfe4b6c 
…t doesn't exist instead, when checking to see if root container exists, check / instead of ->root because ->root is auto prepended
@benliles
Moved checkout for an object in fopen to where it returns a stream fo…
d26eb1d 
…r it when reading to remove null return when object doesn't exist
@benliles
Added a large 'free space' return value since 0 restricts uploading a…
d8965a7 
…nything in the interface, not an easy way to get free space with swift
@benliles
Added container specific stat code
98a5d0b
@benliles
Folder removing through unlink now calls rmdir instead of failing
810e133
@benliles
Updated gitignore
790a428
@benliles
Merge branch 'master' of github.com:benliles/core
3c9f1f8
@benliles
Fixed db quote issue with postgresql
af8b6b3
@icewind1991
Copy link
Contributor

Very nice, there is a lot of other changes in this though, could you update your pull request to only include the changes to apps/files_external/lib/swift.php

@benliles
Copy link
Contributor Author

Sorry about that, merged from owncloud/core master as I was doing the pull request.

@benliles benliles closed this Oct 10, 2012
@jozefcernak jozefcernak mentioned this pull request Sep 22, 2013
Closed
@Kaelber Kaelber mentioned this pull request May 9, 2016
Closed
@sermilrod sermilrod mentioned this pull request Jun 25, 2016
Closed
@osadmin osadmin mentioned this pull request Aug 16, 2016
Merged
@tonybluem tonybluem mentioned this pull request Sep 13, 2016
Closed
@stephan-l stephan-l mentioned this pull request Nov 4, 2016
Closed
@TDannhauer TDannhauer mentioned this pull request Nov 30, 2016
Closed
@fsantiago07044 fsantiago07044 mentioned this pull request Feb 9, 2017
Closed
@happiness-now happiness-now mentioned this pull request Apr 22, 2017
Closed
@montanaviking montanaviking mentioned this pull request Apr 28, 2017
Closed
@andriusign andriusign mentioned this pull request Dec 12, 2017
Closed
@popsy700 popsy700 mentioned this pull request Apr 17, 2018
Closed
@CristianCardosoA CristianCardosoA mentioned this pull request Aug 8, 2018
Closed
@lock lock bot locked as resolved and limited conversation to collaborators Aug 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@benliles @icewind1991 @bartv2 @DeepDiver1975