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

Crashes when try create index for database without tables #1293

Closed
5 of 14 tasks
angkastiel opened this issue Jan 14, 2018 · 4 comments
Closed
5 of 14 tasks

Crashes when try create index for database without tables #1293

angkastiel opened this issue Jan 14, 2018 · 4 comments
Assignees
Labels

Comments

@angkastiel
Copy link
Task lists! Give feedback

@angkastiel angkastiel commented Jan 14, 2018

Details for the issue

  1. Create new database (File->New Database), type name in save dialog, press 'Save' button.
  2. 'Edit table definition' dialog appears. Click on 'Cancel' button.
  3. Create index (Edit->Create Index)
  4. Application crashes

Useful extra information

I'm opening this issue because:

  • DB4S is crashing
  • DB4S has a bug
  • DB4S needs a feature
  • DB4S has another problem

I'm using DB4S on:

  • Windows: ( _version: 10 x64 )
  • Linux: ( distro: ___ )
  • Mac OS: ( version: ___ )
  • Other: ___

I'm using DB4S version:

  • 3.10.1
  • 3.10.0
  • 3.9.1
  • Other: 3.10.99 (nightly build at 13 jan 2018)

I have also:

@justinclift
Copy link
Member

@justinclift justinclift commented Jan 14, 2018

Thanks @angkastiel. Well spotted. 😄

MKleusberg added a commit that referenced this issue Jan 15, 2018
Fix some crashes in the Edit Index dialog which happen if you try to add
a new index when there are no tables in the database yet.

See issue #1293.
@MKleusberg
Copy link
Member

@MKleusberg MKleusberg commented Jan 15, 2018

Thanks, @angkastiel. The crash should be fixed in the next nightly build. Can you check if it's working for you? 😃

@angkastiel
Copy link
Author

@angkastiel angkastiel commented Jan 16, 2018

@MKleusberg, i checked nightly build at 16 jan 2018. The application no longer crashes. Thanks! 😄

@angkastiel angkastiel closed this Jan 16, 2018
@justinclift
Copy link
Member

@justinclift justinclift commented Jan 16, 2018

Excellent. Thanks for verifying @angkastiel. smile:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants