{"thread":{"id":"30326","subject":"[PATCH] gitweb: Don't set owner if got empty value from projects.list","startedAt":"2012-04-24T17:50:05Z","lastAt":"2012-04-24T17:50:05Z","messageCount":1,"participants":["Kacper Kornet"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"189977","messageId":"20120424175005.GD15600@camk.edu.pl","threadId":"30326","inReplyTo":null,"subject":"[PATCH] gitweb: Don't set owner if got empty value from projects.list","fromName":"Kacper Kornet","fromEmail":"draenog@pld-linux.org","sentAt":"2012-04-24T17:50:05Z","receivedAt":"2012-04-24T17:50:05Z","isPatch":true,"sender":{"key":"draenog@pld-linux.org","avatar":"https://avatars.githubusercontent.com/u/608762?v=4"},"body":"Prevent setting owner to an empty value if it is not specified in\nprojects.list file. Otherwise it stops retrieving information about the\nowner from other files.\n\nSigned-off-by: Kacper Kornet <draenog@pld-linux.org>\n---\nThe change is necessary due to 14b289bdf7c31176dc525cbe81173ec636598782:\n\"gitweb: Refactor checking if part of project info need filling\", which\nreplaced !defined with !exists in check if owner is set.  \n\n gitweb/gitweb.perl |    6 ++++--\n 1 file changed, 4 insertions(+), 2 deletions(-)\n\ndiff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl\nindex 3aeeb8b..adb0caa 100755\n--- a/gitweb/gitweb.perl\n+++ b/gitweb/gitweb.perl\n@@ -2906,9 +2906,11 @@ sub git_get_projects_list {\n \t\t\t}\n \t\t\tif (check_export_ok(\"$projectroot/$path\")) {\n \t\t\t\tmy $pr = {\n-\t\t\t\t\tpath => $path,\n-\t\t\t\t\towner => to_utf8($owner),\n+\t\t\t\t\tpath => $path\n \t\t\t\t};\n+\t\t\t\tif ($owner) {\n+\t\t\t\t\t$pr->{'owner'} = to_utf8($owner);\n+\t\t\t\t}\n \t\t\t\tpush @list, $pr;\n \t\t\t}\n \t\t}\n-- \n1.7.10\n\n-- \n  Kacper Kornet\n"}]}