Closed (fixed)
Project:
Drupal core
Version:
7.x-dev
Component:
node system
Priority:
Critical
Category:
Bug report
Assigned:
Reporter:
Created:
30 Mar 2008 at 06:48 UTC
Updated:
21 May 2008 at 11:00 UTC
Jump to comment: Most recent file
If I remember correctly from the Usuability panel at DrupalCon Boston, it was 100% of users who thought 'Content Types' should be under Site Building. I strongly agree.
These patches change all references to 'admin/content/types' to 'admin/build/types'.
This is my first ever patch submission for any software project. I've probably broken all kinds of standards & ettiquette, so please be patient and let me know the bets way to do it in the future. I figured I'd learn by doing. :-)
path.inc_.patch is for the file in /includes. Everything else should be obvious enough.
| Comment | File | Size | Author |
|---|---|---|---|
| #18 | build.patch | 1.05 KB | catch |
| #15 | content_types_remaining.patch | 2.89 KB | recidive |
| #10 | build-types-241021-9.patch | 14.09 KB | floretan |
| #9 | build-types-1.patch | 14.18 KB | keith.smith |
| #4 | build-types.patch | 14.16 KB | matt2000 |
Comments
Comment #1
matt2000 commentedoops. Did I make my patch files wrong? Maybe these are better...
Comment #2
catchOK all the individual patches look fine. The only thing is that they're individual patches instead of one big one.
Depending on what you're using, you should be able to do
cvs diff -up > yourpatch.patchfrom the root of the drupal directory - then you get one big patch file with all the changes to each file. If you use Tortoise CVS or similar, it'll also roll patch files for all changes from the root.This makes it a lot easier to read and test, since you only have to look through one thing (and only have to apply one patch to test it). http://drupal.org/patch/create has everything you need. If you checked out Drupal 7 via cvs, then you may just be able to re-roll with your existing changes.
Massive +1 from me. Thanks!
Comment #3
dries commentedI think this is an important change. We should also revisit the terminology as "content type" was highly confusing, but that could be left for another patch.
Comment #4
matt2000 commentedOK, here it is as one patch file. Thanks for the lesson.
As for the terminology, 'Content Types' seems natural to me, and I can't think of anything better; but then, I've been working with Drupal for a year. Did the test participants have any suggestions?
The term that's been difficult for my clients is 'page.' To most people, every refresh of their browser generates a "page." The theme is the look of the page. They might want a 'page' that lists their articles (which is usually a View).
I often create a generic content type called 'post,' and to them, a 'page' is made up of one post, or a list of posts. They don't understand taxonomy listings as a 'page (generically) of pages (the drupal content type).'
Finally, I noticed that the term 'taxonomy' has returned to replace 'categories' in the admin menu. Was this an intentional decision?
Comment #5
catchPatch looks lovely, thanks for re-rolling. I'll try to test it a bit later today.
I don't think there were any suggestions about terminology from participants (we didn't speak to any of them directly ourselves) - but 'page' 'story' 'content' 'content type' 'create content' and 'post' caused a lot of problems. Story has been changed to 'article' in D7, I started a discussion in the usability group about the general issue of what to call everything else.
However like Dries suggests, we should try to move the menu item asap, and open another issue for any terminology changes.
Taxonomy change from categories is here.
Comment #6
floretan commentedPatch applies cleanly and replaces all instances of admin/content/types. Menu item is moved after cache is cleared.
Comment #7
keith.smith commentedWhile reviewing this patch, I notice that there is an instance of "Story" that should have been changed to "Article". I've created a patch for that at #241021: Reference to "Story" in help text not renamed., rather than rolling it into this one. I can reroll #241021: Reference to "Story" in help text not renamed. after this gets committed.
Comment #8
dries commentedI applied this patch but it fails at one chunk. It will need to get rerolled.
Comment #9
keith.smith commentedRerolled. The hunk failed due to the renaming of that one last instance of "Story" to "Article".
Comment #10
floretan commentedRe-rolled patch to adapt to the change from #241021: Reference to "Story" in help text not renamed.. Just changed Story to Article in the patch directly and it applies cleanly. No other changes, so marking as RTBC.
Comment #11
dries commentedI think a recent patch might have broken this patch again. Sorry.
Comment #12
floretan commentedRe-tested the patch on a current checkout from HEAD, and it works fine (applies cleanly with both the patch command and Eclipse).
Comment #13
catchhttp://drupal.org/cvs?commit=109329
yay!
Comment #14
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.
Comment #15
recidive commentedThe applied patch was incomplete. Editing and deleting content types are still on 'admin/content'.
Patch to correct this attached.
Comment #16
dries commentedGood catch. Committed to CVS HEAD. Thanks.
Comment #17
webchickThis patch broke translation.test (at least). Please fix.
Comment #18
catchtranslation.test was reviewed on April 10th, after the original patch here was committed. So the test should really have been failing due to the bad path in the code and passed when the followup patch went in rather than vice versa.
I can't find admin/content/node-type anywhere else in core, so hopefully this one will fix it.
Comment #19
catchSince the patch is trivial, and all translation tests pass with it applied, I'm going to mark my own patch RTBC.
Comment #20
catchMarking back to closed in favour of: http://drupal.org/node/260499#comment-851924