What are the Slack Archives?

It’s a history of our time together in the Slack Community! There’s a ton of knowledge in here, so feel free to search through the archives for a possible answer to your question.

Because this space is not active, you won’t be able to create a new post or comment here. If you have a question or want to start a discussion about something, head over to our categories and pick one to post in! You can always refer back to a post from Slack Archives if needed; just copy the link to use it as a reference..

Hello community, after a recent update (spryker/product-management 0.19.16 > 0.19.17) the product

s.kroll
s.kroll Head of Technology Spryker @ basecom Posts: 120 🪐 - Explorer
edited March 2021 in Help

Hello community,
after a recent update (spryker/product-management 0.19.16 > 0.19.17) the product-management GUI in zed stopped working. The ajax-request which fetches the table-data fails with this error:

Cannot determine the column to bind to the parameter in clause "SpyProductAbstractLocalizedAttributes.fkLocale = ?".

I kinda tracked it down to this line:
https://github.com/spryker/product-management/blob/master/src/Spryker/Zed/ProductManagement/Communication/Table/ProductTable.php#L147

If changed from fkLocale to fk_locale (like the database-column-name) it works. Is it a known problem or do I have do update other dependencies to get it back working again?

Comments

  • Stanislav Matveyev
    Stanislav Matveyev Tech Lead @ Spryker Sprykee Posts: 211 🧑🏻‍🚀 - Cadet

    what database type do you use?

  • s.kroll
    s.kroll Head of Technology Spryker @ basecom Posts: 120 🪐 - Explorer

    Hey @UKBTMFXMJ, on our local dev-setups we use the docker/sdk where postgres is configured. I did not tested the behaviour in another environment so far

  • Stanislav Matveyev
    Stanislav Matveyev Tech Lead @ Spryker Sprykee Posts: 211 🧑🏻‍🚀 - Cadet

    It's regression issue after latest release, internal ticket is created. We will fix a new patch release soon. Thanks for the reporting.

  • s.kroll
    s.kroll Head of Technology Spryker @ basecom Posts: 120 🪐 - Explorer

    Thanks for confirmation. We currently fixed the version to 0.19.16 but will update to the new release when available

  • Stanislav Matveyev
    Stanislav Matveyev Tech Lead @ Spryker Sprykee Posts: 211 🧑🏻‍🚀 - Cadet

    Ok, we will let you know in this thread.

  • s.kroll
    s.kroll Head of Technology Spryker @ basecom Posts: 120 🪐 - Explorer

    Great, thank you 🙂

  • Valerii Trots
    Valerii Trots SRE @ Spryker Sprykee Posts: 1,654 ✨ - Novice

    Just out of the curiosity, where exactly the problem appears?
    Just checked using public spryker-shop/b2c-demo-shop. Updated mentioned module to the latest with dependencies and have no problems in Zed. Am I looking at wrong place?

  • s.kroll
    s.kroll Head of Technology Spryker @ basecom Posts: 120 🪐 - Explorer

    The problem occured on one of our projects (initially started on the 202009.0 release from b2c-suite) after we did a composer update to get the latest patches/fixes, @valerii.trots.
    This composer updated did an update for spryker/product-management 0.19.16 to 0.19.17.
    Afterwards, the table-ajaxcall on /product-management fails with the exception mentioned above. Reverting back to 0.19.16 immediately resolves this. Also a temporary change in the vendor-file (see my initial post) resolves this.
    As far as I can see we have no project-level changes in pyz/Zed/ProductManagement

  • Valerii Trots
    Valerii Trots SRE @ Spryker Sprykee Posts: 1,654 ✨ - Novice

    Interesting. 🤔

    docker/sdk cli composer update spryker/product-management
Finished: success: 1, skipped: 0, failure: 0, total: 1
Updating spryker/product-management (0.19.7 => 0.19.12): Loading from cache

    docker/sdk cli composer update spryker/product-management --with-dependencies
Finished: success: 66, skipped: 0, failure: 0, total: 66
Updating spryker/product-management (0.19.12 => 0.19.17)

    And there are no problems for me locally.
    But ok, if Stanislav confirms the issue, then it will be fixed.

    I was just curious. 🙂

  • Stanislav Matveyev
    Stanislav Matveyev Tech Lead @ Spryker Sprykee Posts: 211 🧑🏻‍🚀 - Cadet

    I'm still checking 🙂

  • s.kroll
    s.kroll Head of Technology Spryker @ basecom Posts: 120 🪐 - Explorer

    ... and I did not want to claim we did everything correct here 😄

  • Dmitriy Krainiy
    Dmitriy Krainiy Sprykee Posts: 3 🧑🏻‍🚀 - Cadet

    Hello @UM0EZB9GT
    Please, share your Propel version.

    Thanks!

  • Dmitriy Krainiy
    Dmitriy Krainiy Sprykee Posts: 3 🧑🏻‍🚀 - Cadet

    and postgres

  • Stanislav Matveyev
    Stanislav Matveyev Tech Lead @ Spryker Sprykee Posts: 211 🧑🏻‍🚀 - Cadet

    Just checked on local docker setup
    with
    DBMS: PostgreSQL (ver. 9.6.21)

    and spryker/propel-orm:1.13.0

    it works with SpyProductAbstractLocalizedAttributes.fkLocale

  • s.kroll
    s.kroll Head of Technology Spryker @ basecom Posts: 120 🪐 - Explorer

    Our setup is:
    Postgres is 9.6.20 (from PG_VERSION env) and propel/propel is fixed on "2.0.0-alpha11" (unchanged since project-setup via
    https://github.com/spryker-shop/b2c-demo-shop.git -b 202009.0)

  • s.kroll
    s.kroll Head of Technology Spryker @ basecom Posts: 120 🪐 - Explorer

    spryker/propel-orm is at 1.12.0 in this specific project

  • Stanislav Matveyev
    Stanislav Matveyev Tech Lead @ Spryker Sprykee Posts: 211 🧑🏻‍🚀 - Cadet

    Ok, thank you very much @UM0EZB9GT. The problem is found. spryker/product-management is incompatible with spryker/propel-orm:1.12.0 (only with 1.13.0)

  • Stanislav Matveyev
    Stanislav Matveyev Tech Lead @ Spryker Sprykee Posts: 211 🧑🏻‍🚀 - Cadet

    will release the fix soon

  • s.kroll
    s.kroll Head of Technology Spryker @ basecom Posts: 120 🪐 - Explorer
    edited March 2021

    Great, thank you

  • Stanislav Matveyev
    Stanislav Matveyev Tech Lead @ Spryker Sprykee Posts: 211 🧑🏻‍🚀 - Cadet

    @UM0EZB9GT please try https://github.com/spryker/product-management/releases/tag/0.19.18

  • s.kroll
    s.kroll Head of Technology Spryker @ basecom Posts: 120 🪐 - Explorer

    Iit works, thank you @UKBTMFXMJ 🙂

Categories