Skip to content

Conversation

@patrioticcow
Copy link

idea:

$usersData = $firebase->get('/users', [
'orderBy'      => 'id',
'startAt'      => '1',
'limitToFirst' => '80',
]);

this is good for pagination and other things.

idea:

$usersData = $firebase->get('/users', [
	'orderBy'      => 'id',
	'startAt'      => '1',
	'limitToFirst' => '80',
]);

this is good for pagination and other things.
@patrioticcow
Copy link
Author

The interface can be modified to add $data to get(). and maybe getString() can be improved

@mastef
Copy link

mastef commented Jan 12, 2016

@ktamas77 this one is even better than the previous PR. Any ideas on merging this?

@marnickmenting
Copy link

While trying to pass a boolean as value this fails, as it is changed to a string. I have added this code to getString(197):
elseif($val === false) $str .= '&' . $key . '=false'; elseif($val === true) $str .= '&' . $key . '=true';

@safizn
Copy link

safizn commented Apr 14, 2016

Hi thanks for applying this must feature. How can I use this with composer instead of the original respiratory ?
I tried:
sudo composer require patrioticcow/firebase-php master
but it gives me error like:

Problem 1
- The requested package patrioticcow/firebase-php could not be found in any
version, there may be a typo in the package name.
Potential causes:

Could you guide me to the correct solution ?


SOLUTION: To use a forked version of a Github repository with Composer follow:
http://stackoverflow.com/questions/13498519/how-to-require-a-fork-with-composer
http://stackoverflow.com/questions/13325158/contributing-to-open-source-bundles-from-vendor-directory/27970559#27970559
https://getcomposer.org/doc/05-repositories.md#vcs

@ktamas77
Copy link
Owner

Extra query parameters can be added from V2.1.0

@ktamas77 ktamas77 closed this Jun 14, 2016
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.

5 participants