Home » Download » Modules » Flexinode » Issues

flexinode 'Help text' is redundant

ahoeben - April 21, 2006 - 11:18
Project:Flexinode
Version:4.7.x-1.x-dev
Component:User interface
Category:task
Priority:normal
Assigned:ahoeben
Status:closed
Description

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

#1

ahoeben - April 17, 2007 - 20:52
Version:HEAD» 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
Login or register to post comments

#2

Bèr Kessels - April 18, 2007 - 19:11

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).

Login or register to post comments

#3

ahoeben - April 18, 2007 - 19:20
Status:active» patch (code 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.patch3.44 KB
Login or register to post comments

#4

ahoeben - April 19, 2007 - 23:28

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

AttachmentSize
flexinode_helptext_0.patch3.44 KB
Login or register to post comments

#5

ahoeben - April 19, 2007 - 23:31

oops. %s

AttachmentSize
flexinode_helptext_1.patch3.43 KB
Login or register to post comments

#6

Bèr Kessels - April 20, 2007 - 07:53
Status:patch (code needs review)» fixed

Applied, tested an committed. Thanks a lot!

Login or register to post comments

#7

Anonymous - May 4, 2007 - 08:01
Status:fixed» closed
Login or register to post comments
 
 

Drupal is a registered trademark of Dries Buytaert.