[Django] #24177: Docs. inspectdb now inspect the database view
Django
-------------------------------+-----------------------
Reporter: luzfcb | Owner: nobody
Type: Uncategorized | Status: new
Component: Documentation | Version: 1.8alpha1
Severity: Normal | Keywords:
Triage Stage: Unreviewed | Has patch: 0
Easy pickings: 1 | UI/UX: 0
-------------------------------+-----------------------
Hello, recently, in the commit
https://github.com/django/django/commit/b8cdc7dcc3fc6897fb2a75f90023f5c67aad327f
support was added for the "database view" in the inspectdb management
command, However, this was not documented.
this pull-request fix this: https://github.com/django/django/pull/3946
--
Ticket URL: <https://code.djangoproject.com/ticket/24177>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
Django
Reporter: luzfcb | Owner: nobody
Type: | Status: new
Cleanup/optimization |
Component: Documentation | Version: 1.8alpha1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 1 | UI/UX: 0
Changes (by MarkusH):
* needs_better_patch: => 1
* needs_tests: => 0
* needs_docs: => 0
* has_patch: 0 => 1
* type: Uncategorized => Cleanup/optimization
* stage: Unreviewed => Accepted
Comment:
PR: https://github.com/django/django/pull/3946
--
Ticket URL: <https://code.djangoproject.com/ticket/24177#comment:1>
Django
Reporter: luzfcb | Owner: nobody
Type: | Status: new
Cleanup/optimization |
Component: Documentation | Version: 1.8alpha1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Easy pickings: 1 | UI/UX: 0
Changes (by timgraham):
* needs_tests: 0 => 1
Comment:
I think we also need to add a test to confirm this feature.
--
Ticket URL: <https://code.djangoproject.com/ticket/24177#comment:2>
Django
Reporter: luzfcb | Owner: nobody
Type: | Status: new
Cleanup/optimization |
Component: Documentation | Version: 1.8alpha1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 1 | Patch needs improvement: 1
Comment (by jarshwah):
Replying to [comment:2 timgraham]:
> I think we also need to add a test to confirm this feature.
A test was added when cross database support was added for Oracle. Fairly
sure this should cover it.
--
Ticket URL: <https://code.djangoproject.com/ticket/24177#comment:3>
Django
Reporter: luzfcb | Owner: timgraham
Type: | Status: assigned
Cleanup/optimization |
Component: Documentation | Version: 1.8alpha1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Ready for
| checkin
Has patch: 1 | Needs documentation: 0
Easy pickings: 1 | UI/UX: 0
Changes (by timgraham):
* owner: nobody => timgraham
* needs_better_patch: 1 => 0
* status: new => assigned
* needs_tests: 1 => 0
* stage: Accepted => Ready for checkin
Comment:
Great, I'll do a light edit on the English and commit this then.
--
Ticket URL: <https://code.djangoproject.com/ticket/24177#comment:4>
Django
Reporter: luzfcb | Owner: timgraham
Type: | Status: assigned
Cleanup/optimization |
Severity: Normal | Resolution:
Keywords: | Triage Stage: Ready for
| checkin
Has patch: 1 | Needs documentation: 0
Easy pickings: 1 | UI/UX: 0
Comment (by luzfcb):
Sorry for the mistakes, english is not my native language. I still have
quite a lot to learn.
I added the corrections suggested by MarkusH.
See: https://github.com/django/django/pull/3946/commits
As this new feature is included, I believe it is necessary to
differentiate a model generated from a table and a model generated from a
view. Add a simple comment "maps a view" (or other text) in the generated
class already solves it.
If, in the future, django offer support for "synonym" feature of Oracle,
differentiate this can be useful.
In my specific case, the company I started working on last mont, has an
Oracle database with at least 67 schemas, and over 4320 tables, views and
synonym. The database of this size, differentiate what is table, what is
view and what is synonym becomes quite useful.
--
Ticket URL: <https://code.djangoproject.com/ticket/24177#comment:5>
Django
Reporter: luzfcb | Owner: timgraham
Cleanup/optimization |
Component: Documentation | Version: 1.8alpha1
Severity: Normal | Resolution: fixed
Keywords: | Triage Stage: Ready for
| checkin
Has patch: 1 | Needs documentation: 0
Easy pickings: 1 | UI/UX: 0
Changes (by Markus Holtermann <info@…>):
* status: assigned => closed
* resolution: => fixed
Comment:
In [changeset:"bd691f4586c8ad45bd059ff9d3621cbf8afdcdce"]:
{{{
#!CommitTicketReference repository=""
revision="bd691f4586c8ad45bd059ff9d3621cbf8afdcdce"
Fixed #24177 -- Added documentation about database view support in
inspectdb
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/24177#comment:6>
Django
Reporter: luzfcb | Owner: timgraham
Type: | Status: closed
Cleanup/optimization |
Severity: Normal | Resolution: fixed
Keywords: | Triage Stage: Ready for
| checkin
Has patch: 1 | Needs documentation: 0
Easy pickings: 1 | UI/UX: 0
Comment (by Markus Holtermann <info@…>):
In [changeset:"e69eea73d94c058728143416058a6257c765a9b8"]:
{{{
#!CommitTicketReference repository=""
revision="e69eea73d94c058728143416058a6257c765a9b8"
[1.8.x] Fixed #24177 -- Added documentation about database view support in
inspectdb
Backport of bd691f4586c8ad45bd059ff9d3621cbf8afdcdce from master
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/24177#comment:7>
Django
Reporter: luzfcb | Owner: timgraham
Type: | Status: closed
Cleanup/optimization |
Severity: Normal | Resolution: fixed
Keywords: | Triage Stage: Ready for
| checkin
Has patch: 1 | Needs documentation: 0
Easy pickings: 1 | UI/UX: 0
Comment (by Tim Graham <timograham@…>):
In [changeset:"9f0d67137c98aa296471e1b7f57ae43f5bb17db6" 9f0d6713]:
{{{
#!CommitTicketReference repository=""
revision="9f0d67137c98aa296471e1b7f57ae43f5bb17db6"
Fixed #25038 -- Reverted incorrect documentation about inspectdb
introspecting views.
This reverts commit bd691f4586c8ad45bd059ff9d3621cbf8afdcdce (refs
#24177).
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/24177#comment:8>
Django
Reporter: luzfcb | Owner: timgraham
Type: | Status: closed
Cleanup/optimization |
Severity: Normal | Resolution: fixed
Keywords: | Triage Stage: Ready for
| checkin
Has patch: 1 | Needs documentation: 0
Easy pickings: 1 | UI/UX: 0
Comment (by Tim Graham <timograham@…>):
In [changeset:"fdaf9359ab84b06da48f3bb0d5a924b62ace0561" fdaf935]:
{{{
#!CommitTicketReference repository=""
revision="fdaf9359ab84b06da48f3bb0d5a924b62ace0561"
[1.8.x] Fixed #25038 -- Reverted incorrect documentation about inspectdb
introspecting views.
This reverts commit e69eea73d94c058728143416058a6257c765a9b8 (refs
#24177).
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/24177#comment:9>