Giter Club home page Giter Club logo

Comments (15)

lsmith77 avatar lsmith77 commented on August 15, 2024

use php composer.phar install dev not update ..

from cmf-sandbox.

jordillonch avatar jordillonch commented on August 15, 2024

I have the same problem:

My sequence of commands are:

git clone git://github.com/symfony-cmf/cmf-sandbox.git
cd cmf-sandbox
cp app/config/parameters.yml.dist app/config/parameters.yml
cp app/config/phpcr_doctrine_dbal.yml.dist app/config/phpcr.yml
curl -s http://getcomposer.org/installer | php --
php composer.phar require jackalope/jackalope-doctrine-dbal:dev-master

from cmf-sandbox.

lsmith77 avatar lsmith77 commented on August 15, 2024

did you revert the composer.lock that was updated?

from cmf-sandbox.

lsmith77 avatar lsmith77 commented on August 15, 2024

ah i see we are actually suggesting to use update in the README .. i will fix that. but when i do that i don't get the issue you are noting. however i am seeing couple issues when loading the fixtures:

  • "General error: 1364 Field 'sort_order' doesn't have a default value"
  • "Item / not found in workspace default"

@dbu and @petesiss

from cmf-sandbox.

lsmith77 avatar lsmith77 commented on August 15, 2024

btw the issue you are noted above is now tracked here schmittjoh/JMSDiExtraBundle#28

hopefully we can figure out what is going wrong with the fixture loading tomorrow.

from cmf-sandbox.

petesiss avatar petesiss commented on August 15, 2024

Strange. Ive just done a fresh install of the sandbox with doctrine dbal, but I'm not getting either of those errors. Fixtures appear to be loading ok for me and sandbox appears to be functional.

Also tested the fixtures on our own project and they seem fine.

The jackalope repo in the composer.lock is one commit behind where it should be for the sandbox, but that commit doesn't seem to make any difference for me on whether the fixtures load.

from cmf-sandbox.

lsmith77 avatar lsmith77 commented on August 15, 2024

I am getting the feeling that this approach of using "require" to add the dependency is too flaky because it ends up doing an "update" afterwards since the new dependency isn't in the "composer.lock" file.

from cmf-sandbox.

petesiss avatar petesiss commented on August 15, 2024

Id not come across "require" until now. If its only purpose is to avoid users downloading libs they may not use, even if it wasn't flakey Id say for the sandbox better to simplify the install instructions for the dbal version by just including it by default.

Could always put a note at the end to suggest users can remove the implementation they're not using if they're worried about having the redundant code.

from cmf-sandbox.

lsmith77 avatar lsmith77 commented on August 15, 2024

yeah .. i will switch to using "require-dev"

from cmf-sandbox.

lsmith77 avatar lsmith77 commented on August 15, 2024

argh .. now i see again why i used the "require" command .. since the midgard dependency requires a php extension. so i am switching things for doctrine dbal only for now. we could of course provide multiple composer.lock files.

maybe @Seldaek has an idea for now to best do this ..

from cmf-sandbox.

Seldaek avatar Seldaek commented on August 15, 2024

@lsmith77 I'm not sure what this is about, but the require command is a bit buggy. It takes the current versions from what's installed instead of the lock file, so when it updates the newly required package, it can rollback stuff (compared to the lock) which isn't too good.

from cmf-sandbox.

Seldaek avatar Seldaek commented on August 15, 2024

Also that means if you have nothing installed, it's basically ignoring the lock file I think.

from cmf-sandbox.

lsmith77 avatar lsmith77 commented on August 15, 2024

@jordillonch i fixed some issues in the Doctrine DBAL to address the fixture loading issues I saw. i have also updated the docs to use php composer.phar install --dev instead of the require.

so please revert any local changes you might have, ensure that the tables are deleted again from your database (optionally use app/console doctrine:phpcr:init:dbal --drop) and then try again.

hope all will work then.

from cmf-sandbox.

lsmith77 avatar lsmith77 commented on August 15, 2024

please reopen if the issue isn't fixed yet

from cmf-sandbox.

jordillonch avatar jordillonch commented on August 15, 2024

Thank you Lukas! Now it's working :)

from cmf-sandbox.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.