Could somebody make a patch for Tasks & Projects

Kamikaze - July 3, 2009 - 09:11
Project:Storm
Version:6.x-1.x-dev
Component:Storm Task
Category:feature request
Priority:normal
Assigned:Unassigned
Status:duplicate
Description

I am impressed with Storm module, but I would like to make a request.

Could someone make a patch that shows "Person/People" column in Tasks?
Also if another can be made, I would like to see "Project Manager" column show in Projects.
That will be great.

PS:Sorry, I am duplicating my requests in different places because I am new to this site.

#1

homoludens - July 3, 2009 - 18:19

basically it is adding new field "Project Manager" to "Storm Project" and adding "Person" to "Storm Task" - and consequently to displayed list?
sounds like nice feature - one could easily find person responsible for specific project and task.

just give me confirmation that i have understood you right, and i will try to make it relatively soon.

what do others think of this?

#2

Magnity - July 3, 2009 - 20:32

If this is to be commited, I think it should be done in conjunction with with #380008: Development of Storm Teammember, if not, then fine.

#3

Kamikaze - July 6, 2009 - 02:30

Homoloudens,
You understood it right. Thanks for working on it.

#4

Kamikaze - July 8, 2009 - 02:21

So, have there been any pregress on this?
Just checking.

#5

homoludens - July 8, 2009 - 11:34

Not sure if we will do it like that right now. Please take a look at discussion #380008: Development of Storm Teammember. We are brainstorming new features of teammember module and it's connection with others modules. I have, already, included your suggestion as idea, but your input can be valuable.

Problem is much bigger then just assigning people to project/tasks.
That does not mean that i wont add those columns before redesign, but i must wait a little to think about it.

#6

homoludens - July 14, 2009 - 12:33
Status:active» needs review

here is patch that adds person to tasks. it is against latest dev (2009-Jul-07).

before you try it you will have to execute this two sql commands:

ALTER TABLE stormtask ADD person_organization_nid INT;
ALTER TABLE stormtask ADD person_nid INT;

try and report.

EDIT: added _nid for name of new table columns.

AttachmentSize
stormtask_add_person_diff_dev.patch 9.86 KB

#7

bamakojeff - July 13, 2009 - 04:06

Thanks for the patch. I just applied it to stormtask 6.x-1.23 and the following part failed.

---------------------------------------------------------
diff -Naur stormtaskOrig/stormtask.module stormtask/stormtask.module
--- stormtaskOrig/stormtask.module 2009-07-07 01:04:15.000000000 +0200
+++ stormtask/stormtask.module 2009-07-08 17:38:07.000000000 +0200

@@ -479,7 +597,7 @@

function stormtask_form_alter(&$form, $form_state, $form_id) {
- if (module_exists('date_api')) {
+ if (module_exists('date_api') && module_exists('date_popup')) {
if ('stormtask_node_form' == $form_id) {
$node = $form['#node'];
$form['group4']['datebegin'] = array(
-----------------------------------------------------------------------

There's no "function stormtask_form_alter" anywhere in the stormtask module files.

#8

homoludens - July 13, 2009 - 11:46

it is patch against -dev. But even if you use -dev simple patching would not work - it is better to edit those changes by hand, because there are a lot of changes in storm task in last few weeks so some lines are moved - so it is hard to make patch.

on the other side you can edit stormtask_add_person_diff_dev.patch file and delete following part, since it is not needed for this patch, and try again:

@@ -479,7 +597,7 @@


function stormtask_form_alter(&$form, $form_state, $form_id) {
-  if (module_exists('date_api')) {
+  if (module_exists('date_api') && module_exists('date_popup')) {
     if ('stormtask_node_form' == $form_id) {
$node = $form['#node'];
$form['group4']['datebegin'] = array(

#9

bamakojeff - July 14, 2009 - 10:18

Thanks for the clarification. I also had to slightly change the mysql commands that needed to be run. The stormtask.module refers to columns "person_nid" and "person_organization_nid" rather than just "person_organization" and "person".

ALTER TABLE stormtask ADD person_organization_nid; INT;
ALTER TABLE stormtask ADD person_nid INT;

Thanks again for the patch. :-)

#10

homoludens - July 14, 2009 - 12:34

Thank you for correction. Glad you find it useful.

#11

bamakojeff - July 24, 2009 - 09:53

If it helps anyone, here's a patch against v.1.23 that allows you to assign a person to a task. I just made a few changes so it worked with this branch and themed the new sections like everything else.

Thanks again for all the work on the Storm module and for taking the time to write this patch so we can assign someone to each task. It makes our life easier. :-)

AttachmentSize
stormtask_add_person_diff_1.23.patch 10.75 KB

#12

Magnity - July 27, 2009 - 12:19
Status:needs review» duplicate

Duplicate topic of #392414: Assign projects, tasks, and tickets to people

 
 

Drupal is a registered trademark of Dries Buytaert.