Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#20967 closed Uncategorized (invalid)

wrong grammar in the docmentation

Reported by: eng.abdelrazzak@… Owned by: anonymous
Component: Documentation Version: 1.5
Severity: Normal Keywords: documentation
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description

in the documentation pdf page 94
in the 3.2 Models and databases > Model inheritance >Multi-table inheritance

the statement "All of the fields of Place will also be available in Restaurant, although the data will reside in a different database table. So these are both possible:"
should be "All of the fields of Place will also be available in Restaurant, although the data will reside in a different database tables. So these are both possible:"

Attachments (1)

20967.diff (541 bytes ) - added by eng.abdelrazzak@… 11 years ago.
the patch of the solution

Download all attachments as: .zip

Change History (7)

comment:1 by eng.abdelrazzak@…, 11 years ago

Owner: changed from nobody to anonymous
Status: newassigned

by eng.abdelrazzak@…, 11 years ago

Attachment: 20967.diff added

the patch of the solution

comment:2 by anonymous, 11 years ago

attached the solution patch

comment:3 by Ahmed Abdel Razzak <eng.abdelrazzak@…>, 11 years ago

Resolution: fixed
Status: assignedclosed

comment:4 by Ahmed Abdel Razzak <eng.abdelrazzak@…>, 11 years ago

Resolution: fixed
Status: closednew

comment:5 by Carl Meyer, 11 years ago

Resolution: invalid
Status: newclosed

Hi, and thanks for the contribution. I think, though, that the existing grammar is correct, and the change in your patch makes it incorrect. "a database tables" is wrong, since there is number disagreement between "a" (singular) and "tables" (plural).

It could be changed to say "resides in different database tables" (eliminating the "a") instead of "resides in a different database table." Either of these is grammatically correct, and really I think either one would semantically correct as well. The current singular version is describing just the Place data, and saying that that data resides in a different table from the table containing Restaurant data.

Since the current text is correct both grammatically and semantically, I don't think there's any need to make a change, so I'm closing this ticket.

Thanks again for taking the time to file a ticket and a patch!

comment:6 by Ahmed Abdel Razzak <eng.abdelrazzak@…>, 11 years ago

a link to the branch containing the fix on github https://github.com/artmees/django/tree/ticket_20967

Note: See TracTickets for help on using tickets.
Back to Top