aboutsummaryrefslogtreecommitdiffstats
path: root/ui-repolist.c
Commit message (Expand)AuthorAgeFilesLines
* ui-repolist,ui-shared: remove redundant title on repo anchors•••The title attribute was being set to the same value as the anchor element text. Signed-off-by: Chris Mayo <aklhfex@gmail.com> Reviewed-by: Eric Wong <e@80x24.org> Reviewed-by: Petr Vorel <petr.vorel@gmail.com> Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com> Chris Mayo2022-12-191-1/+1
* ui-repolist: do not return unsigned (negative) value•••The function read_agefile() returns time_t, which is a signed datatime. We should not return unsigned (negative) value here. Reported-by: Johannes Stezenbach <js@linuxtv.org> Signed-off-by: Christian Hesse <mail@eworm.de> Christian Hesse2019-11-221-1/+1
* global: make 'char *path' const where possible•••Signed-off-by: Christian Hesse <mail@eworm.de> Christian Hesse2019-06-051-1/+1
* global: remove functionality we deprecated for cgit v1.0•••The man page states these were deprecated for v1.0. We are past v1.1, so remove the functionality. Signed-off-by: Christian Hesse <mail@eworm.de> Reviewed-by: John Keeping <john@keeping.me.uk> Christian Hesse2018-06-271-3/+0
* html: html_ntxt with no ellipsis•••For implementing a ui-blame page, there is need for a function that outputs a selection from a block of text, transformed for HTML output, but with no further modifications or additions. Signed-off-by: Jeff Smith <whydoubt@gmail.com> Reviewed-by: John Keeping <john@keeping.me.uk> Jeff Smith2017-10-031-1/+2
* Remove unused variable from sort_section()•••Signed-off-by: Lukas Fleischer <lfleischer@lfos.de> Lukas Fleischer2017-04-051-1/+0
* ui-repolist: properly sort by age•••When empty repos exist, comparing them against an existing repo with a good mtime might, with particular qsort implementations, not sort correctly, because of this brokenness: if (get_repo_modtime(r1, &t) && get_repo_modtime(r2, &t)) However, sorting by the age column works as expected, so anyway, to tidy things up, we simply reuse that function. Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com> Jason A. Donenfeld2017-03-301-21/+17
* css: highlight even table rows and skip empty rows•••This is stolen from kernel.org css [0]. [0] https://git.kernel.org/cgit-korg-0.10.1.css Christian Hesse2016-11-231-1/+1
* ui-repolist: fix memory leakChristian Hesse2016-10-121-1/+3
* ui-repolist: fix memory leakChristian Hesse2016-10-071-1/+4
* ui: show ages in the originator's timezone•••This affects the tooltip showing the full time and the case when a date is sufficiently old to be shown in full rather than as an offset. Signed-off-by: John Keeping <john@keeping.me.uk> John Keeping2016-02-081-1/+1
* ui-shared: remove "format" from cgit_print_age()•••We never use any format other than FMT_SHORTDATE, so move that into the function. Signed-off-by: John Keeping <john@keeping.me.uk> John Keeping2016-02-081-1/+1
* ui-repolist: return HTTP 404 if no repositories found•••Return HTTP status code 404 Not found when querying a non-existent repository, which signals to search engines that a repository no longer exists. Further, some webservers such as nginx permit logging requests to different files depending on the HTTP code. Signed-off-by: Peter Colberg <peter@colberg.org> Peter Colberg2016-01-131-3/+17
* ui-repolist: extract repo visibility criteria to separate function•••Signed-off-by: Peter Colberg <peter@colberg.org> Peter Colberg2016-01-131-3/+10
* ui-repolist: initialize char *buf to NULL•••readfile() can fail if the agefile is not readable. Make sure free() does not free an ininitialized string. Signed-off-by: Christian Hesse <mail@eworm.de> Christian Hesse2016-01-131-1/+1
* ui-repolist: fix resource leak: free allocation from cgit_currenturl•••Coverity-id: 13930 Signed-off-by: Christian Hesse <mail@eworm.de> Christian Hesse2015-10-091-1/+3
* ui-repolist: fix resource leak: free before return•••Coverity-id: 13931 Signed-off-by: Christian Hesse <mail@eworm.de> Christian Hesse2015-10-091-1/+3
* about: move layout into page functions•••Signed-off-by: John Keeping <john@keeping.me.uk> John Keeping2015-08-141-1/+4
* Remove redundant includes•••These are all included in git-compat-util.h (when necessary), which we include in cgit.h. Signed-off-by: John Keeping <john@keeping.me.uk> John Keeping2015-08-131-1/+0
* log: allow users to follow a file•••Teach the "log" UI to behave in the same way as "git log --follow", when given a suitable instruction by the user. The default behaviour remains to show the log without following renames, but the follow behaviour can be activated by following a link in the page header. Follow is not the default because outputting merges in follow mode is tricky ("git log --follow" will not show merges). We also disable the graph in follow mode because the commit graph is not simplified so we end up with frequent gaps in the graph and many lines that do not connect with any commits we're actually showing. We also teach the "diff" and "commit" UIs to respect the follow flag on URLs, causing the single-file version of these UIs to detect renames. This feature is needed only for commits that rename the path we're interested in. For commits before the file has been renamed (i.e. that appear later in the log list) we change the file path in the links from the log to point to the old name; this means that links to commits always limit by the path known to that commit. If we didn't do this we would need to walk down the log diff'ing every commit whenever we want to show a commit. The drawback is that the "Log" link in the top bar of such a page links to the log limited by the old name, so it will only show pre-rename commits. I consider this a reasonable trade-off since the "Back" button still works and the log matches the path displayed in the top bar. Since following renames requires running diff on every commit we consider, I've added a knob to the configuration file to globally enable/disable this feature. Note that we may consider a large number of commits the revision walking machinery no longer performs any path limitation so we have to examine every commit until we find a page full of commits that affect the target path or something related to it. Suggested-by: RenĂ© Neumann <necoro@necoro.eu> Signed-off-by: John Keeping <john@keeping.me.uk> John Keeping2015-08-121-1/+1
* ui-repolist: make sortcolumn definitions 'static const'•••These are not used outside this file and are not declared; they are also never modified. Signed-off-by: John Keeping <john@keeping.me.uk> John Keeping2015-03-091-2/+2
* Avoid non-ANSI function declarations•••Sparse says things like: warning: non-ANSI function declaration of function 'calc_ttl' Signed-off-by: John Keeping <john@keeping.me.uk> John Keeping2015-03-091-3/+3
* Make root handling sane again.Jason A. Donenfeld2015-03-031-1/+1
* ui-shared: Add current url helper function.Jason A. Donenfeld2015-03-031-2/+2
* ui-repolist: use ctx.qry.url instead of rooturl, in case we're filteringJason A. Donenfeld2015-03-031-2/+2
* ui-repolist: use correct owner query linkJason A. Donenfeld2015-03-031-1/+1
* Add repo.hide and repo.ignore•••These options can be used to hide a repository from the index or completely ignore a repository, respectively. They are particularly useful when used in combination with scan-path. Signed-off-by: Lukas Fleischer <cgit@cryptocrack.de> Lukas Fleischer2015-01-291-0/+2
* repolist: add owner-filter•••This allows custom links to be used for repository owners by configuring a filter to be applied in the "Owner" column in the repository list. Chris Burroughs2014-12-231-7/+13
* git: update to v2.2.1•••Update to git version v2.2.1, including API changes. Signed-off-by: Christian Hesse <mail@eworm.de> Christian Hesse2014-12-231-3/+4
* git: update for git 2.0•••prefixcmp() and suffixcmp() have been remove, functionality is now provided by starts_with() and ends_with(). Retrurn values have been changed, so instead of just renaming we have to fix logic. Everything else looks just fine. Christian Hesse2014-06-281-1/+1
* repolist: make owner clickable to search•••Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com> Jason A. Donenfeld2014-01-171-0/+6
* Switch to exclusively using global ctx•••Drop the context parameter from the following functions (and all static helpers used by them) and use the global context instead: * cgit_print_http_headers() * cgit_print_docstart() * cgit_print_pageheader() Remove context parameter from all commands Drop the context parameter from the following functions (and all static helpers used by them) and use the global context instead: * cgit_get_cmd() * All cgit command functions. * cgit_clone_info() * cgit_clone_objects() * cgit_clone_head() * cgit_print_plain() * cgit_show_stats() In initialization routines, use the global context variable instead of passing a pointer around locally. Remove callback data parameter for cache slots This is no longer needed since the context is always read from the global context variable. Signed-off-by: Lukas Fleischer <cgit@cryptocrack.de> Lukas Fleischer2014-01-171-3/+3
* filter: return on null filter from open and close•••So that we don't have to include the if(filter) open_filter(filter) block everywhere, we introduce the guard in the function itself. This should simplify quite a bit of code. Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com> Jason A. Donenfeld2014-01-141-4/+2
* ui-repolist: HTML-escape cgit_rooturl() response•••This is for consistency with other callers. The value returned from cgit_rooturl is not guaranteed to be HTML-safe. Signed-off-by: John Keeping <john@keeping.me.uk> John Keeping2014-01-121-1/+3
* filter: pass extra arguments via cgit_open_filter•••This avoids poking into the filter data structure at various points in the code. We rely on the fact that the number of arguments is fixed based on the filter type (set in cgit_new_filter) and that the call sites all know which filter type they're using. Signed-off-by: John Keeping <john@keeping.me.uk> John Keeping2014-01-121-7/+3
* Update copyright information•••* Name "cgit Development Team" as copyright holder to avoid listing every single developer. * Update copyright ranges. Signed-off-by: Lukas Fleischer <cgit@crytocrack.de> Lukas Fleischer2014-01-081-2/+1
* ui-summary: Pass filename to about-filter•••This gives the about-filter API the same semantics as source-filter, where the filter receives the filename so it can decide what to do next with it. While we're at it, plug a memory leak. Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com> Jason A. Donenfeld2013-05-251-2/+6
* use struct strbuf instead of static buffers•••Use "struct strbuf" from Git to remove the limit on file path length. Notes on scan-tree: This is slightly involved since I decided to pass the strbuf into add_repo() and modify if whenever a new file name is required, which should avoid any extra allocations within that function. The pattern there is to append the filename, use it and then reset the buffer to its original length (retaining a trailing '/'). Notes on ui-snapshot: Since write_archive modifies the argv array passed to it we copy the argv_array values into a new array of char* and then free the original argv_array structure and the new array without worrying about what the values now look like. Signed-off-by: John Keeping <john@keeping.me.uk> John Keeping2013-04-081-12/+16
* Always #include corresponding .h in .c files•••While doing this, remove declarations from header files where the corresponding definition is declared "static" in order to avoid build errors. Also re-order existing headers in ui-*.c so that the file-specific header always comes immediately after "cgit.h", helping with future consistency. Signed-off-by: John Keeping <john@keeping.me.uk> John Keeping2013-04-081-0/+1
* Convert pager navigation into a unordered list•••It is common practice and semantically appropriate to use unordered lists for long navigation lists. This also fixes the layout of very long pager navigations in Webkit-based browsers. Signed-off-by: Lukas Fleischer <cgit@cryptocrack.de> Lukas Fleischer2013-03-201-2/+4
* Mark several functions/variables static•••Spotted by parsing the output of `gcc -Wmissing-prototypes [...]`. Signed-off-by: Lukas Fleischer <cgit@cryptocrack.de> Lukas Fleischer2013-03-041-7/+7
* White space around control verbs.•••Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com> Jason A. Donenfeld2013-03-041-2/+2
* Fix several whitespace errors•••* Remove whitespace at the end of lines. * Replace space indentation by tabs. * Add whitespace before/after several operators ("+", "-", "*", ...) * Add whitespace to assignments ("foo = bar;"). * Fix whitespace in parameter lists ("foobar(foo, bar, 42)"). Signed-off-by: Lukas Fleischer <cgit@cryptocrack.de> Lukas Fleischer2013-03-041-3/+3
* Make "owner" column on index page configurable•••This is not really needed for personal sites where all repos belong to the same person. Since it is pretty useful for shared sites however, it should be configurable. Signed-off-by: Florian Pritz <bluewind@xinu.at> Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com> Florian Pritz2013-02-011-7/+12
* ui-repolist: Add "section-sort" flag to control section sorting.•••Flag which, when set to "1", will sort the sections on the repository listing by name. Set this flag to "0" if the order in the cgitrc file should be preserved. Default value: "1". Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com> Tobias Bieniek2012-10-171-1/+1
* ui: Remember to print ampersand as proper html entities.William Bell2012-10-171-1/+1
* ui-repolist: Rename section-sort to repository-sort.•••Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com> Jason A. Donenfeld2012-10-161-1/+1
* ui-repolist: do not use agefile if it's date could not be parsed•••Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl> Ferry Huberts2012-10-091-2/+4
* ui-repolist: Bold the currently viewed page.•••Signed-off-by: Jamie Couture <jamie.couture@gmail.com> Jamie Couture2012-10-081-4/+7
* Update copyright headers to have latest dates.Jason A. Donenfeld2012-07-121-0/+1