diff options
| author | Alexandru DAMIAN <alexandru.damian@intel.com> | 2014-02-13 13:36:54 +0000 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2014-03-09 12:23:57 -0700 |
| commit | 7008a24792704b8e757c583663636addf49f397d (patch) | |
| tree | 85af5fc40963aabf6f243242da761fe6ad94a14e /bitbake/lib | |
| parent | 4ae598195b60ea5a4c0a3efaded679208e1cc07e (diff) | |
| download | poky-7008a24792704b8e757c583663636addf49f397d.tar.gz | |
bitbake: toastergui: fix task executed status display
The proper way to get a display-able value
in Django is to use a model method instead of clutter
the template with if/else.
[YOCTO #5641]
(Bitbake rev: bb21b71dab70db163b804c7ebf27b85c59a39112)
Signed-off-by: Alexandru DAMIAN <alexandru.damian@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'bitbake/lib')
| -rw-r--r-- | bitbake/lib/toaster/orm/models.py | 5 | ||||
| -rw-r--r-- | bitbake/lib/toaster/toastergui/templates/recipe.html | 6 | ||||
| -rw-r--r-- | bitbake/lib/toaster/toastergui/templates/tasks.html | 8 |
3 files changed, 7 insertions, 12 deletions
diff --git a/bitbake/lib/toaster/orm/models.py b/bitbake/lib/toaster/orm/models.py index c77df2243e..f96da9c339 100644 --- a/bitbake/lib/toaster/orm/models.py +++ b/bitbake/lib/toaster/orm/models.py | |||
| @@ -116,6 +116,11 @@ class Task(models.Model): | |||
| 116 | def get_related_setscene(self): | 116 | def get_related_setscene(self): |
| 117 | return Task.objects.related_setscene(self) | 117 | return Task.objects.related_setscene(self) |
| 118 | 118 | ||
| 119 | def get_executed_display(self): | ||
| 120 | if self.task_executed: | ||
| 121 | return "Executed" | ||
| 122 | return "Not Executed" | ||
| 123 | |||
| 119 | build = models.ForeignKey(Build, related_name='task_build') | 124 | build = models.ForeignKey(Build, related_name='task_build') |
| 120 | order = models.IntegerField(null=True) | 125 | order = models.IntegerField(null=True) |
| 121 | task_executed = models.BooleanField(default=False) # True means Executed, False means Not/Executed | 126 | task_executed = models.BooleanField(default=False) # True means Executed, False means Not/Executed |
diff --git a/bitbake/lib/toaster/toastergui/templates/recipe.html b/bitbake/lib/toaster/toastergui/templates/recipe.html index eba15baad3..1adb012189 100644 --- a/bitbake/lib/toaster/toastergui/templates/recipe.html +++ b/bitbake/lib/toaster/toastergui/templates/recipe.html | |||
| @@ -115,11 +115,7 @@ | |||
| 115 | <i class="icon-question-sign get-help hover-help" title="This task fetches the source code"></i> | 115 | <i class="icon-question-sign get-help hover-help" title="This task fetches the source code"></i> |
| 116 | </td> | 116 | </td> |
| 117 | 117 | ||
| 118 | {% if task.task_executed %} | 118 | <td><a {{ task|task_color }} href="{% url "task" build.pk task.pk %}">{{task.get_executed_display}}</a></td> |
| 119 | <td><a {{ task|task_color }} href="{% url "task" build.pk task.pk %}">Executed</a></td> | ||
| 120 | {% else %} | ||
| 121 | <td><a {{ task|task_color }} href="{% url "task" build.pk task.pk %}">Prebuilt</a></td> | ||
| 122 | {% endif %} | ||
| 123 | 119 | ||
| 124 | <td><a {{ task|task_color }} href="{% url "task" build.pk task.pk %}">{{task.get_outcome_display}}</a></td> | 120 | <td><a {{ task|task_color }} href="{% url "task" build.pk task.pk %}">{{task.get_outcome_display}}</a></td> |
| 125 | <td> | 121 | <td> |
diff --git a/bitbake/lib/toaster/toastergui/templates/tasks.html b/bitbake/lib/toaster/toastergui/templates/tasks.html index 6831f106c0..3479e7e1e7 100644 --- a/bitbake/lib/toaster/toastergui/templates/tasks.html +++ b/bitbake/lib/toaster/toastergui/templates/tasks.html | |||
| @@ -30,13 +30,7 @@ | |||
| 30 | <a href="{%url "task" build.pk task.pk%} ">{{task.task_name}}</a> | 30 | <a href="{%url "task" build.pk task.pk%} ">{{task.task_name}}</a> |
| 31 | </td> | 31 | </td> |
| 32 | <td class="executed"> | 32 | <td class="executed"> |
| 33 | <a href="{%url "task" build.pk task.pk%} "> | 33 | <a href="{%url "task" build.pk task.pk%} ">{{task.get_executed_display}}</a> |
| 34 | {% if task.task_executed %} | ||
| 35 | Executed | ||
| 36 | {% else %} | ||
| 37 | Not executed | ||
| 38 | {% endif %} | ||
| 39 | </a> | ||
| 40 | </td> | 34 | </td> |
| 41 | <td class="outcome"> | 35 | <td class="outcome"> |
| 42 | <a href="{%url "task" build.pk task.pk%} ">{{task.get_outcome_display}}</a> | 36 | <a href="{%url "task" build.pk task.pk%} ">{{task.get_outcome_display}}</a> |
