Cant get to work with Laravel 5.4 #36
Comments
|
Clean install of Laravel 5.4 ErrorException in Connection.php line 20: What needs to be resolved? |
|
Hi @1jason1 , Thank you for using this library. I have made some changes in Connection.php to resolve the getConfig() problem. Can you please retry it by installing fresh copy. Also the project is still in development mode so you may get unexpected bugs while using. Let me know if this solved your problem. |
|
Notes from clean install:
Generating optimized class loader
The compiled services file has been removed.
php artisan key:generate
Application key [base64:OvqjnxvpRHYNmQXph0jHhv9Tfr2TX3inO1MZ1W6Ym6w=] set successfully.
Application ready! Build something amazing.
jasonf /srv/www$ cd test2
jasonf /srv/www/test2$ composer require sgpatil/oriquent
Using version v5.4.0.1 for sgpatil/oriquent
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Package operations: 2 installs, 0 updates, 0 removals
- Installing sgpatil/orientdb-php (v0.0.3) Downloading: 100%
- Installing sgpatil/oriquent (v5.4.0.1) Downloading: 100%
Writing lock file
Generating optimized autoload files
Warning: Ambiguous class resolution, "Sgpatil\Orientphp\Command\CreateClass" was found in both "/srv/www/test2/vendor/sgpatil/orientdb-php/src/Sgpatil/Orientphp/Command/CreateClass.php" and "/srv/www/test2/vendor/sgpatil/orientdb-php/src/Sgpatil/Orientphp/Command/MakeClass.php", the first will be used.
Illuminate\Foundation\ComposerScripts::postUpdate
php artisan optimize
Generating optimized class loader
The compiled services file has been removed.
Also when trying to register a new user via http://test2.dev/register
Form is displayed no problem, after submitting the form get the following error.
[cid:[email protected]]
On Apr 5, 2017, at 8:20 AM, Sumit Patil <[email protected]<mailto:[email protected]>> wrote:
Hi @1jason1<https://github.com/1jason1> , Thank you for using this library. I have made some changes in Connection.php to resolve the getConfig() problem. Can you please retry it by installing fresh copy. Also the project is still in development mode so you may get unexpected bugs while using. Let me know if this solved your problem.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#36 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AFUe08NYPqfZwnsdDKFJ85EgfMXAIQrQks5rs5UjgaJpZM4MyNNS>.
|
|
Hello @1jason1 , can you please try branch dev-master as I have made changes in that branch only. |
|
Installed dev-master.
Error after submitting the register form.
[cid:03A73769-63B3-47F3-BE82-7979D0A590FA]
On Apr 6, 2017, at 2:46 AM, Sumit Patil <[email protected]<mailto:[email protected]>> wrote:
Hello @1jason1<https://github.com/1jason1> , can you please try branch dev-master as I have made changes in that branch only.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#36 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AFUe01N_trGrmLKVo0SOTWdF2OaC2V-Sks5rtJhWgaJpZM4MyNNS>.
|
|
I’ve had to take a break from this now picking it back up. Still having issues getting it to work.
I have a valid user created but when I try to login I get the following error, please help.
[cid:3403BC8C-89D5-44BF-85F2-0F3470A219D5]
[cid:24630D22-0AB8-481F-BC2B-BFDCD51753D0]
On Apr 6, 2017, at 9:21 PM, Jason Forgey <[email protected]<mailto:[email protected]>> wrote:
Installed dev-master.
Error after submitting the register form.
<PastedGraphic-3.png>
On Apr 6, 2017, at 2:46 AM, Sumit Patil <[email protected]<mailto:[email protected]>> wrote:
Hello @1jason1<https://github.com/1jason1> , can you please try branch dev-master as I have made changes in that branch only.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#36 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AFUe01N_trGrmLKVo0SOTWdF2OaC2V-Sks5rtJhWgaJpZM4MyNNS>.
|
|
Hi @1jason1 , Actually I have tried the same code on fresh installation and everything is running properly in my system. Can you please tell me your system configuration like laravel version, Orientdb version, OS, Server, php version, screen shot of error message etc. This may help me to figure out the problem. |
|
Thanks for the response.
Here are my settings.
* orientdb-community-2.2.17
* Laravel Framework 5.4.19
* MacOS (Sierra) 10.12.4
* Apache via MAMP Pro v4.1.1
* PHP 7.1.1
**Snip of my app.php**
/*
* Package Service Providers...
*/
Laravel\Tinker\TinkerServiceProvider::class,
Sgpatil\Orientdb\OrientdbServiceProvider::class,
**This is my User Model:**
namespace App;
use Illuminate\Notifications\Notifiable;
use Illuminate\Auth\Authenticatable;
use Sgpatil\Orientdb\Eloquent\Model AS Model;
use Illuminate\Auth\Passwords\CanResetPassword;
use Illuminate\Foundation\Auth\Access\Authorizable;
use Illuminate\Contracts\Auth\Authenticatable as AuthenticatableContract;
use Illuminate\Contracts\Auth\Access\Authorizable as AuthorizableContract;
use Illuminate\Contracts\Auth\CanResetPassword as CanResetPasswordContract;
class User extends Model implements
AuthenticatableContract,
AuthorizableContract,
CanResetPasswordContract
{
use Notifiable, Authenticatable, Authorizable, CanResetPassword;
/**
* The attributes that are mass assignable.
*
* @var array
*/
protected $fillable = [
'name', 'email', 'password',
];
/**
* The attributes that should be hidden for arrays.
*
* @var array
*/
protected $hidden = [
'password', 'remember_token',
];
}
**Here is the error I get when trying to register a new user**
ErrorException in Builder.php line 14:Declaration of Sgpatil\Orientdb\Eloquent\Builder::has($relation, $operator = '>=', $count = 1, $boolean = 'and', ?Sgpatil\Orientdb\Eloquent\Closure $callback = NULL) should be compatible with Illuminate\Database\Eloquent\Builder::has($relation, $operator = '>=', $count = 1, $boolean = 'and', ?Closure $callback = NULL)
|
|
Also, I get the following Warning when doing a Composer Update and at the point where composer is "Generating optimized autoload files" I get this Warning.
|
|
I was also having this issue but it looks like the updates on dev-master fixed the issue. As @1jason1 mentioned, there is also the composer Ambiguous class resolution warning. Should we open a new issue for this? |
|
@rickvschmidt How did you get the errors to go away for Laravel 5.4? Not sure if you follow my config setup in prior posts but do I have my configs setup correctly? Are you able to register users and login via the auth package that Laravel provides by default? In my composer.json in my require section I'm using the following, is this correct to pull the version your using? |
|
@1jason1 Not exactly yet, though I must admit that I am new to this package altogether. I have the same composer config as you do. I at least got past the original error you got while viewing the default Laravel view. I am currently having issues running any migrations... for me it is complaining that the database doesn't exist, while I am able to make a new model via Artisan orient:create |
|
@sgpatil I am unable to run any migrations. orient:install works fine, and is able to write to the DB, I am also able to call orient:create without issue. However, when I try to run a migration, I get the following error with the same configuration file setup... so I know the database exists and the drivers are able to connect. This running your latest dev-master branch and I am also on Laravel 5.4
|
|
I simply get [ReflectionException] |
|
@rickvschmidt, Is your .env file is configured properly ? It should be like DB_CONNECTION=orientdb |
|
@sgpatil Yes, I have changed it. For some reason it requires Root access in order to properly work (not sure if this is by design, though I find it strange that admin access is not adequate). Once everything is switched over to root, the database not found error goes away, but then I get the error that @1jason1 gets (first post in this thread) when trying to use a basic auth page produced by Laravel. It appears there are some updates to be done to make the query builder work with Laravel 5.4. |
|
So I've tried all I can to get this driver to work with Laravel 5.4, I really like OrientDb vs Neo4j. I have a large social media type project that would work better under OrientDb but there just doesn't seem to be the right support for drivers yet for Laravel. Can anyone point me in the direction of another driver option before I have to go the Neo4j route? |
|
Sorry @1jason1, Unfortunately there are no perfect orientdb driver for laravel. You may contribute to this to build that one, PRs are always welcome. Due to ongoing projects I am not able to give full time to this project. Even I also wanted such a driver for one of my project which I have built using Neo4j (NeoEloquent Driver). Hope to build it better. |
|
@sgpatil I didn't mean to come off rude in my comment so I apologize if I did and thank you for the work you have done on this. I would like to help out, do you have any documentation on a recommend path to making the driver? |
ErrorException in Builder.php line 14:
Declaration of Sgpatil\Orientdb\Eloquent\Builder::has($relation, $operator = '>=', $count = 1, $boolean = 'and', ?Sgpatil\Orientdb\Eloquent\Closure $callback = NULL) should be compatible with Illuminate\Database\Eloquent\Builder::has($relation, $operator = '>=', $count = 1, $boolean = 'and', ?Closure $callback = NULL)
How do I get this to work, was using Neo4j but I want to try out OrientDb
The text was updated successfully, but these errors were encountered: