summaryrefslogtreecommitdiff
path: root/lib/banzai
Commit message (Collapse)AuthorAgeFilesLines
* Re-allow `name` attribute on user-provided anchor HTMLRobert Speicher2017-09-221-1/+2
|
* Eliminate N+1 queries referencing issuessh-project-feature-eager-loadStan Hu2017-09-181-1/+2
| | | | | | | | To load issue 1, we see that in #38033 that about 835 ms of the SQL queries were due to loading ProjectFeature. We should be able to cut this down by eagerly loading this information. Closes #38033
* Remove ImageLazyLoadFilter from EmailPipeline37629-lazy-image-loading-breaks-notification-mails-for-an-added-screenshotRobert Speicher2017-09-122-1/+8
|
* Merge branch 'rs-issue-36104' into 'security-9-5'Douwe Maan2017-09-071-0/+3
| | | | | [9.5] Disallow the `name` attribute on all user-provided markup See merge request gitlab/gitlabhq!2166
* Merge branch 'rs-issue-36098' into 'security-9-5'Robert Speicher2017-09-071-1/+21
| | | | | [9.5] Limit `style` attribute on `th` and `td` elements to specific properties See merge request gitlab/gitlabhq!2155
* Merge branch 'anakashima/gitlab-ce-fix_wiki_toc_indent' into 'master'Sean McGivern2017-09-061-18/+72
|\ | | | | | | | | Wiki table of contents are now properly nested to reflect header level See merge request !13909
| * Refactor table of contents buildingRobert Speicher2017-09-061-19/+11
| |
| * Refactor `HeaderNode#find_parent`Robert Speicher2017-09-061-6/+7
| | | | | | | | | | Now the instance variable assignment is all contained within `#initialize`, and the `children.push` behavior is more explicit.
| * Refactor TableOfContentsFilter's nested table of contentsRobert Speicher2017-09-061-42/+64
| | | | | | | | Most of the logic is now self-contained within the `HeaderNode` class.
| * Fix indentation level in Wiki's TOC items to regard header levelAkihiro Nakashima2017-09-061-4/+43
| |
* | Adds cacheless render to Banzai object render34509-improves-markdown-rendering-performance-for-commits-listTiago Botelho2017-09-064-8/+16
| |
* | Adds CommitDescriptionRendererTiago Botelho2017-09-061-0/+7
|/
* Merge branch 'rs-more-public-send-whitelists' into 'master'Rémy Coutable2017-08-164-6/+6
|\ | | | | | | | | Whitelist or fix additional `Gitlab/PublicSend` cop violations See merge request !13467
| * Whitelist or fix additional `Gitlab/PublicSend` cop violationsrs-more-public-send-whitelistsRobert Speicher2017-08-144-6/+6
| | | | | | | | | | An upcoming update to rubocop-gitlab-security added additional violations.
* | Enable Layout/TrailingWhitespace cop and auto-correct offensesRobert Speicher2017-08-151-2/+2
|/
* DRY up caching in AbstractReferenceFilterSean McGivern2017-08-071-40/+20
| | | | | We had a lot of copied and pasted code, when the different elements were very small and easy to get wrong.
* Support references to group milestonesSean McGivern2017-08-072-11/+64
| | | | | Group milestones can only be referred to by name, not IID. They also do not support cross-project references.
* Don't bother going through an entire Banzai pipeline for empty textrs-minor-banzai-perfRobert Speicher2017-08-031-0/+2
| | | | | | | | | | This bails out of `cacheless_render` immediately unless the provided text is present, since there's no point. This is a slight improvement in our test performance. Across the creation of 1,000 `Namespace` records, which caches the `description` field and which is blank by default in its factory, this saves about four seconds, which... sure. Why not.
* Rename more path_with_namespace -> full_path or disk_pathGabriel Mazetto2017-08-011-1/+1
|
* Rename many path_with_namespace -> full_pathGabriel Mazetto2017-08-012-3/+3
|
* Resolve "Lazy load images on the Frontend"Tim Zallmann2017-07-245-2/+20
|
* Support both internal and external issue trackersJarka Kadlecova2017-07-242-4/+8
|
* Merge branch 'acet-fix-tooltip-placement' into 'master'Jacob Schatz2017-07-061-0/+2
|\ | | | | | | | | | | | | Fix markdown preview tooltip cut off issue Closes #31748 See merge request !12367
| * Banzai: Add tooltip attributes to reference filter.acet-fix-tooltip-placementFatih Acet2017-07-031-0/+2
| |
* | Create and use project path helpers that only need a project, no namespaceDouwe Maan2017-07-057-8/+7
|/
* Improve support for external issue referencesadam-external-issue-references-spikeAdam Niedzielski2017-06-304-49/+5
|
* Enable Style/DotPosition Rubocop :cop:Grzegorz Bizon2017-06-213-6/+6
|
* Don't return nil for missing objects from parser cachedm-fix-parser-cacheDouwe Maan2017-06-141-1/+1
|
* Merge branch '25934-project-snippet-vis' into 'security-9-2'DJ Mountney2017-06-089-10/+14
| | | | | | Fix visibility when referencing snippets See merge request !2101
* Fix /unsubscribe slash command creating extra todos33032-invalid-you-directly-addressed-yourself-todo-when-using-unsubscribeSean McGivern2017-05-301-2/+3
| | | | | | | | | | | | | | | | | | | | | | The /unsubscribe slash command means that we check if the current user is subscribed to the issuable without having an explicit subscription. That means that we use the UserParser to find references to them in the notes. The UserParser (and all parsers inheriting from BaseParser) use RequestStore to cache ActiveRecord objects, so that we don't need to load the User object each time, if we're parsing references a bunch of times in the same request. However, it was always returning _all_ of the previously cached items, not just the ones matching the IDs passed. This would mean that we did two runs through with UserParser if you were mentioned in a comment, and then mentioned someone else in your comment while using /unsubscribe: 1. Because /unsubscribe was used, we see if you were mentioned in any comments. 2. Because you mentioned someone, we find them - but we would also get back your user, even if you didn't mention yourself. This would have the effect of creating a mention or directly addressed todo for yourself incorrectly. The fix is simple: only return values from the cache matching the IDs passed.
* Merge branch '29852-latex-formatting' into 'master'Robert Speicher2017-05-243-0/+31
|\ | | | | | | | | | | | | Fix LaTeX formatting for AsciiDoc wiki Closes #29852 See merge request !11212
| * Fix LaTeX formatting for AsciiDoc wikiJarka Kadlecova2017-05-243-0/+31
| |
* | Catch all URI errors in ExternalLinkFilterdm-catch-uri-errorsDouwe Maan2017-05-181-1/+1
|/
* Merge branch 'bvl-security-9-1-validate-urls-in-markdown-using-uri'Robert Speicher2017-05-101-18/+18
| | | | | | (security-9-1) Add correct `rel` attributes to external links when rendering markdown See merge request !2097
* Merge branch 'bvl-security-9-1-markup-pipeline'Robert Speicher2017-05-101-0/+13
| | | | | | (security-9-1) Render asciidoc & other markup using banzai in a pipeline See merge request !2098
* Backport of multiple_assignees_feature [ci skip]Valery Sizov2017-05-041-1/+1
|
* Remove N+1 queries when checking nodes visible to userfix-n-plus-one-project-featuresSean McGivern2017-05-021-1/+2
| | | | | | | N in this case is low, as it's the number of distinct projects referenced from MRs, rather than the number of MRs referenced (issues use their own optimization). Still, on issues or MRs which are often referenced from MRs, it will save a few queries.
* Merge branch 'blackst0ne/gitlab-ce-fix_emoji_parser'Sean McGivern2017-04-271-1/+4
|\
| * Fix rendering emoji inside a stringblackst0ne2017-04-251-1/+4
| |
* | issues_for_nodes => issuables_for_nodes31280-skip-issueables-without-projectLin Jen-Shin2017-04-261-3/+3
| |
* | Follow feedback on the reviewLin Jen-Shin2017-04-261-4/+4
| |
* | Skip issuable without a project in IssuableExtractor#extractLin Jen-Shin2017-04-251-1/+5
| | | | | | | | Closes #31280
* | Start versioning cached markdown fieldsNick Thomas2017-04-211-14/+7
| |
* | Remove a use of module_functionNick Thomas2017-04-211-12/+10
| |
* | Merge branch 'issuable-state-custom-links' into 'master' Sean McGivern2017-04-201-2/+2
|\ \ | | | | | | | | | | | | | | | | | | Do not append issuable state to links with custom anchor Closes #30916 See merge request !10782
| * | Do not append issuable state to links with custom anchorissuable-state-custom-linksAdam Niedzielski2017-04-191-2/+2
| | | | | | | | | | | | | | | Closes #30916 Change the presentation from [closed] to (closed)
* | | Merge branch 'plantuml-filter-after-highlight' into 'master' Sean McGivern2017-04-201-4/+4
|\ \ \ | |/ / |/| | | | | | | | | | | | | | Fix PlantUML integration in GFM Closes #30557 See merge request !10651
| * | Fix PlantUML integration in GFMplantuml-filter-after-highlightAdam Niedzielski2017-04-191-4/+4
| | | | | | | | | | | | | | | Fixes #30557 Use "lang" attribute instead of "class" in PlantumlFilter.
* | | Display issuable state only in notes and issuable descriptionAdam Niedzielski2017-04-191-0/+2
|/ / | | | | | | Closes #30964
* | Fix regression in rendering Markdown references that do not existsh-fix-base-parserStan Hu2017-04-151-1/+2
| | | | | | | | Closes #30972
View Lorry Controller Status