Simple ToDo List with CodeIgniter!

[eluser]Horvat[/eluser]
Here is simple 'app' that I made w/CI.
Would like to hear some critics
https://github.com/Horki/ci_todo

[eluser]solid9[/eluser]
I use todoist.com lol

[eluser]Horvat[/eluser]
What are you trying to say?

[eluser]solid9[/eluser]
Do you have live demo of your work?
Or can you make a live demo.
I'm a little lazy installing it.

[eluser]Horvat[/eluser]
you have 'pm'

[eluser]solid9[/eluser]
I tried it.
Not bad for a beginner.

But try using todoist.com and compare your work.

I have question.
Why re-invent the wheel?
Unless you are doing this for practice for purpose only.

[eluser]Horvat[/eluser]
Well I can do more complex stuff.
But I need a task, so I'm searching for tasks on internet and do it my way with CI,
If you have any idea, please let me know

PS I'm trying to 'pump' my github so I can get a job as php developer

[eluser]solid9[/eluser]
Lets wait for others to criticize your work.

[eluser]jmadsen[/eluser]
Use joins instead of:

Code:

foreach($all_list as $list) {
        $username = $this->model_users->get_user_by_id($list->users_id);
        $data['all_list']['list']['user_id'][] = $username->id;
        $data['all_list']['list']['username'][] = $username->username;
        $data['all_list']['list']['list_id'][] = $list->id;
        $data['all_list']['list']['title'][] = $list->title;
        $priority = $this->model_priority->priority_by_id($list->priority_id);
        $data['all_list']['list']['priority'][] = $priority->level;
        $data['all_list']['list']['date_added'][] = $list->date_added;
        $data['all_list']['list']['date_complete'][] = $list->date_complete;
        $data['all_list']['list']['status'][] = $list->status;
      }

Also study how to use MY_Model & MY_Controllers

Code:

$options = array(
    1 => 'High',
    2 => 'Medium',
    3 => 'Low'
  );
  
  for($i = 1; $i <= 31; $i++) {
    $day[$i] = $i;
  }
  
  for($i = 1; $i <= 12; $i++) {
    $month[$i] = $i;
  }
  
  $this_year = date('Y');
  for($i = $this_year; $i <= $this_year+20; $i++) {
    $year[$i] = $i;
  }

Do that in your controllers,models or a helper, not your views

[eluser]Horvat[/eluser]
Tnx!
PS
Any idea for next task?