Bug 80938 - view not editable
Summary: view not editable
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Database (show other bugs)
Version: 4.3.0.1 rc
Hardware: Other All
: medium blocker
Assignee: Caolán McNamara
QA Contact:
URL:
Whiteboard: bibisected target:4.4.0.0.beta2 targe...
Keywords: bisected, regression
: 83776 (view as bug list)
Depends on:
Blocks:
 
Reported: 2014-07-05 07:55 UTC by Dominik
Modified: 2014-12-04 23:54 UTC (History)
5 users (show)

See Also:
i915 platform:
i915 features:


Attachments
example database (3.33 KB, application/vnd.oasis.opendocument.database)
2014-07-05 17:37 UTC, Terrence Enger
Details

Description Dominik 2014-07-05 07:55:01 UTC
neither via edit nor via edit in SQL view it's possible to edit (or even view) the statement of an existing view.

reproduce:
create a table with 1 column
create a view that shows that 1 colum
after saving, try to reopen the edit from for the view

reproducible: always

seen at least in 4.3.0.2 and 4.4.0.0
Comment 1 Terrence Enger 2014-07-05 17:37:38 UTC
Created attachment 102306 [details]
example database

I observe:

(A) with embedded hsqldb (try1_hsqldb.odb attached):

    (1) "Edit in SQL view..." on the view displays the expected
        window, but the pane for the SQL statement is entirely empty.
        This pane should show a sql SELECT statement.

    (2) "Edit" on the view displays the Design View window, but the
        tables pane is empty and the grid of columns selected into the
        query is empty.  The tables pane should show table "example1"
        and the grid should show two fields selected: "number" and
        "words".

(B) with embedded firebird:

    (1) There is no menu option "Edit it SQL view..." in the pop-up
        menu for the view.

    (2) "Edit" on the view displays a window like "Create Table in
        Design View", i.e. grid of fields, the columns being "Field
        Name", "Field Type", and "Description".  All fields in the
        grid are protected.


These observations are from master commit dc795cb, fetched 2014-07-02
2357 UTC, configured:
    --enable-option-checking=fatal
    --enable-dbgutil
    --enable-crashdump
    --without-system-postgresql
    --without-myspell-dicts
    --with-extra-buildid
    --without-doxygen
    --with-external-tar=/home/terry/lo_hacking/git/src
built and running on debian-wheezy 64-bit.
Comment 2 Terrence Enger 2014-07-09 20:24:52 UTC
Correcting comment 1:  The master version that I fetched on 2014-07-02 is 397362d.
Comment 3 Terrence Enger 2014-07-14 17:52:01 UTC
Bibisect tells me:

    ...
        source-hash-c84f0dae40b6f3e64a265efcc76b1757edbdbf45
        ....

and `git bisect log`:

    # bad: [423a84c4f7068853974887d98442bc2a2d0cc91b] source-hash-c15927f20d4727c3b8de68497b6949e72f9e6e9e
    # good: [65fd30f5cb4cdd37995a33420ed8273c0a29bf00] source-hash-d6cde02dbce8c28c6af836e2dc1120f8a6ef9932
    git bisect start 'latest' 'oldest'
    # good: [e02439a3d6297a1f5334fa558ddec5ef4212c574] source-hash-6b8393474974d2af7a2cb3c47b3d5c081b550bdb
    git bisect good e02439a3d6297a1f5334fa558ddec5ef4212c574
    # good: [4850941efe43ae800be5c76e1102ab80ac2c085d] source-hash-980a6e552502f02f12c15bfb1c9f8e6269499f4b
    git bisect good 4850941efe43ae800be5c76e1102ab80ac2c085d
    # bad: [a900e72b6357882284c5955bdf939bf14269f5fb] source-hash-dd1050b182260a26a1d0ba6d0ef3a6fecc3f4e07
    git bisect bad a900e72b6357882284c5955bdf939bf14269f5fb
    # good: [e1d0365cd2b073a859f59ad0a4584385a66dc611] source-hash-2eea96c702a44ab009743b0d22ef639127f0b57b
    git bisect good e1d0365cd2b073a859f59ad0a4584385a66dc611
    # skip: [8f55938c891ee3e4c252b193dba9419f130537bc] source-hash-93f3f72d18e551c8edd6a010cb78d9cbe404f8ef
    git bisect skip 8f55938c891ee3e4c252b193dba9419f130537bc
    # good: [7518fcaf863962bf4f6f3cdf84f6e42f0f59225f] source-hash-ab1f5eab4830f00dbbd7c883b98b59975ecd3bb1
    git bisect good 7518fcaf863962bf4f6f3cdf84f6e42f0f59225f
    # good: [56a3b3c781fc2eb55f46641d89a866a91119a8a3] source-hash-21e6fd2b2dfdb806db320f699e434e6f2351a7b6
    git bisect good 56a3b3c781fc2eb55f46641d89a866a91119a8a3
    # bad: [0b79394752f7ecbab6ab4ecedbfab8551c6e9fbd] source-hash-381613916d42a1e18e2824b5d41028dcfe19659a
    git bisect bad 0b79394752f7ecbab6ab4ecedbfab8551c6e9fbd
    # bad: [5874347fb1093d87d810e030af88327f0dc26a81] source-hash-a3f0803059cd22ddf8590db992183008dedd0fdb
    git bisect bad 5874347fb1093d87d810e030af88327f0dc26a81
    # bad: [a03a074d7d24b285022244e1939b1abbedc76d1e] source-hash-c84f0dae40b6f3e64a265efcc76b1757edbdbf45
    git bisect bad a03a074d7d24b285022244e1939b1abbedc76d1e
Comment 4 Helmut Lohkamp 2014-08-28 15:25:46 UTC
same problem faced here today. Both windows of the views are empty (Edit & SQL).
But running the view it shows the expected data. Also the report using the view(s) run correct. I'd need to change some things in the view but can't right now.
Comment 5 robert 2014-09-11 19:47:12 UTC
*** Bug 83776 has been marked as a duplicate of this bug. ***
Comment 6 Luzius Auer 2014-10-16 10:32:26 UTC
Why is this regression ignored in several recent releases 4.3 and 4.4 ?
It prevents me from upgrading from 4.2.x to V 4.3, as I often have to modify existing views in databases I use to manage a music school.
Therefore I consider this as a blocker, at least for me.
Comment 7 Andras Timar 2014-11-28 15:00:28 UTC
bisected, first bad commit is:
commit d60392fb5b387175dbd2e575848993c02c56922f
Author: Caolán McNamara <caolanm@redhat.com>
Date:   Thu Mar 6 12:05:24 2014 +0000

    coverity#706316 help out coverity re tortured logic
    
    Change-Id: I6d6a948fdac4442c69972316275349fe73b9e741
Comment 8 Commit Notification 2014-12-01 14:08:38 UTC
Caolán McNamara committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=6269b62b525ec22471db56015ac2daed813ec5ff

Resolves: fdo#80938 fix SQL view to be editable

It will be available in 4.5.0.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.
Comment 9 Commit Notification 2014-12-01 14:10:11 UTC
Caolán McNamara committed a patch related to this issue.
It has been pushed to "libreoffice-4-4":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=14b8adadb475aa15a0b9e96b446ec74cec4e5f99&h=libreoffice-4-4

Resolves: fdo#80938 fix SQL view to be editable

It will be available in 4.4.0.0.beta2.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.
Comment 10 Commit Notification 2014-12-02 13:33:55 UTC
Caolán McNamara committed a patch related to this issue.
It has been pushed to "libreoffice-4-3":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=11b4de26833558b482e28619afe09c2c0acd7bb2&h=libreoffice-4-3

Resolves: fdo#80938 fix SQL view to be editable

It will be available in 4.3.6.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.
Comment 11 Commit Notification 2014-12-04 14:03:41 UTC
Caolán McNamara committed a patch related to this issue.
It has been pushed to "libreoffice-4-3-5":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=efaefdf1ca3b03a1db0a96c8a705595009263665&h=libreoffice-4-3-5

Resolves: fdo#80938 fix SQL view to be editable

It will be available in 4.3.5.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.