Download & Extend
Flexinode » Issues

flexinode 'Help text' is redundant

Posted by ahoeben on April 21, 2006 at 11:18am
Project:Flexinode
Version:4.7.x-1.x-dev
Component:User interface
Category:task
Priority:normal
Assigned:ahoeben
Status:closed (fixed)

Issue Summary

The 'Help text' field in admin/node/type/N and the 'admin/settings/content-types/flexinode-1' in admin/settings/content-types/flexinode-N are overlapping functionality.

For clarity, the latter should be removed (or it should mirror the text of admin/node/type/N).

Login or register to post comments

Comments

#1

Posted by ahoeben on April 17, 2007 at 8:52pm
Version:master» 4.7.x-1.x-dev
Assigned to:Anonymous» ahoeben

Subtasks:

  • remove help text box from 'create/edit content-type' pages
  • database update:
    • move data from {flexinode_type}.help to variables named 'flexinode-N_help' for each flexinode type
    • alter the {flexinode_type} table

#2

Posted by Bèr Kessels on April 18, 2007 at 7:11pm

I would prefer to remove that field alltogether. This is a rather large task, because it needs database changes and an upgrade path (so that people don't loose the help texts when upgrading).

#3

Posted by ahoeben on April 18, 2007 at 7:20pm
Status:active» needs review

Something like the attached patch should do it, though I have not yet tested the update script (I only have production sites at the moment ;-)

AttachmentSize
flexinode_helptext.patch 3.44 KB

#4

Posted by ahoeben on April 19, 2007 at 11:28pm

Tested the database update. Turned out I missed a %d... Fixed.

AttachmentSize
flexinode_helptext_0.patch 3.44 KB

#5

Posted by ahoeben on April 19, 2007 at 11:31pm

oops. %s

AttachmentSize
flexinode_helptext_1.patch 3.43 KB

#6

Posted by Bèr Kessels on April 20, 2007 at 7:53am
Status:needs review» fixed

Applied, tested an committed. Thanks a lot!

#7

Posted by Anonymous on May 4, 2007 at 8:01am
Status:fixed» closed (fixed)
nobody click here