laravel-er-diagram-generator icon indicating copy to clipboard operation
laravel-er-diagram-generator copied to clipboard

PHP 8.0 support

Open davi5e opened this issue 3 years ago • 19 comments

PHP 8 compatibility isn't included in the composer.json.

davi5e avatar Dec 04 '20 20:12 davi5e

Also doctrine/dbal 3.0 please :)

peterjaap avatar Dec 08 '20 04:12 peterjaap

I love to see PHP 8 compatibility

lablnet avatar Dec 20 '20 02:12 lablnet

php 8 version throwing error.

sajendramanandhar avatar Feb 22 '21 15:02 sajendramanandhar

+1 to PHP 8 support.

penagos avatar Mar 27 '21 17:03 penagos

php 8 and laravel 8 support please.

humanware avatar Mar 27 '21 22:03 humanware

I was able to get this to work on PHP8 and Laravel 8 by ignoring composer’s language version check, but native support would be nice.

penagos avatar Mar 27 '21 22:03 penagos

Can confirm that PR #94 works :+1:

{
    "repositories": [
        {
            "url": "https://github.com/ali-alharthi/laravel-er-diagram-generator",
            "type": "github"
        }
    ],
    "require": {
        "beyondcode/laravel-er-diagram-generator": "dev-master"
   }
}

tennox avatar May 14 '21 18:05 tennox

Support for PHP 8 please.

CristianH033 avatar Jun 30 '21 12:06 CristianH033

@mpociot and team, could we have some PHP 8 love here?

eugenefvdm avatar Aug 25 '21 08:08 eugenefvdm

8 !!

MenC-Didam avatar Aug 30 '21 12:08 MenC-Didam

@mpociot and team, The latest release doesn't support php version 8. I hope the following change in composer.json will help here "require": { "php": "^7.3|^8.0", } Can it be added in anyway and support php version 8?

prasun125 avatar Sep 13 '21 16:09 prasun125

Update on this would be great!

bernhardh avatar Nov 21 '21 11:11 bernhardh

as @penagos suggest you can get this work by ignoring version check and -W flag

composer require beyondcode/laravel-er-diagram-generator --dev --ignore-platform-reqs -W

mohamed-okasha avatar Jan 07 '22 15:01 mohamed-okasha

Still does not work:

  • beyondcode/laravel-er-diagram-generator 1.5.0 requires doctrine/dbal ~2.3 -> found doctrine/dbal[2.3.0-BETA1, ..., 2.13.x-dev] but it conflicts with your root composer.json require (^3.2).

klimenttoshkov avatar Jan 18 '22 14:01 klimenttoshkov

Yes please <3

Lootfi avatar Mar 22 '22 16:03 Lootfi

Seems like the devs don't care about this, they haven't said a thing.

@beyondcode please do something about this!

Kyngo avatar Apr 11 '22 07:04 Kyngo

project seems abandoned, nothing stops you from forking and making the adjustments.

sietse85 avatar Apr 11 '22 07:04 sietse85

which er generator shall we choose to continue this work? I vote for a separate org, with contributors from all current forks :)

https://useful-forks.github.io/?repo=beyondcode/laravel-er-diagram-generator

this needs to live!

allanlaal avatar May 18 '22 17:05 allanlaal

Just want to share that this PR #102 solved my problem as my repo use PHP 8. For anyone who wish to use this branch to enable laravel-er-diagram-generator in their codebase, this is the steps I used:

  1. Edit your composer.json to install this package but using this PR branch as source. Please note that the comments shouldn't be actually entered into your compose.json as json does not actually support commenting
    // .. bunch of other entries ...
    // add this new `repositories` section if its not presents yet
    "repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/laravel-shift/laravel-er-diagram-generator"
        }
    ],
    "require-dev": {
          // insert this line into require-dev section. of course you could also insert it into regular `require` section if you prefer
        "beyondcode/laravel-er-diagram-generator": "dev-l9-compatibility",
        // ... bunch of other packages...
    },
    // .. bunch of other entries ...
  1. run composer update beyondcode/laravel-er-diagram-generator to install all laravel-er-diagram-generator dependencies.
  2. After this you should already able to run this package. See README file for further information.

my references:

  • https://stackoverflow.com/a/13500676/2496217

notes:

  • just realized that there are already different answer suggesting similar workaround above

azophy avatar Sep 14 '22 06:09 azophy

+1 to PHP 8 support.

seek-x2y avatar Nov 12 '22 05:11 seek-x2y

so current replacement for THIS is:

https://github.com/laravel-shift/laravel-er-diagram-generator/tree/l9-compatibility

allanlaal avatar Nov 17 '22 20:11 allanlaal