Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wrong table name for ABI when eosio::singleton used #93

Closed
Mattijah opened this issue Jan 23, 2023 · 3 comments · Fixed by #125
Closed

Wrong table name for ABI when eosio::singleton used #93

Mattijah opened this issue Jan 23, 2023 · 3 comments · Fixed by #125
Assignees
Labels
bug Something isn't working 👍 lgtm test-refinement

Comments

@Mattijah
Copy link

The following ticket (bug) has been closed despite it not even being resolved and it can still be reproduced in the latest 3.2.0. The bug completely prevents from uploading a contract. More info: EOSIO/eosio.cdt#280

@stephenpdeos
Copy link
Member

@Mattijah can you clarify the version number of CDT you are encountering this? Could you also include an example?

@Mattijah
Copy link
Author

  • The version number is mentioned in my comment above (3.2.0-dev), but since it wasn't fixed since 1.8 it probably is not working in any version.
  • Example can be found in the link attached. Copy:
class [[eosio::contract("test")]] test : public contract {
    public:
        using contract::contract;
        
    [[eosio::action]]
        void whatever() {};

    struct [[eosio::table]] tbl_config {
        uint64_t y;
        uint64_t x;
    };
    
    typedef eosio::singleton<"config"_n, tbl_config> config;
};

Will produce:

"tables": [
        {
            "name": "tbl_config", // <<< wrong
            "type": "tbl_config",
            "index_type": "i64",
            "key_names": [],
            "key_types": []
        }

@stephenpdeos
Copy link
Member

@Mattijah 3.2.0 is prob the leap version. Most recent cdt version is 3.1.0. Thanks for the example.

@stephenpdeos stephenpdeos added the bug Something isn't working label Jan 26, 2023
@mikelik mikelik self-assigned this Apr 3, 2023
@BenjaminGormanPMP BenjaminGormanPMP moved this from Todo to In Progress in Team Backlog Apr 4, 2023
@mikelik mikelik moved this from In Progress to Awaiting Review in Team Backlog Apr 5, 2023
@github-project-automation github-project-automation bot moved this from Awaiting Review to Done in Team Backlog Apr 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working 👍 lgtm test-refinement
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants