10527 Commits

Author SHA1 Message Date
David Ostrovsky
a8a303c809 Fix javadoc unresolved link warning
This change fixes this warning:

  warning - Tag @link: reference not found: PatchSetAncestor

Change-Id: I999749ea8a95c71ee9c3d51a169e698299db243a
2014-02-01 15:11:51 +01:00
Shawn Pearce
0eca733594 Merge "Allow plugins to display errors in the Gerrit ErrorDialog" 2014-01-31 22:51:07 +00:00
David Ostrovsky
42f66e2bd0 Replace GET /changes/<id> call with ChangeApi.get() call in acceptance tests
Change-Id: I84d78bfe5bd4dd038c8f45be9fb3b436b8d9dba1
2014-01-31 22:34:55 +00:00
Shawn Pearce
6ba8448cb8 Merge topic 'plugin-api'
* changes:
  Add ChangeApi.get() method
  Consistently apply camel case naming convention in server code
  Plugin API: Add ChangeStatus
2014-01-31 22:34:36 +00:00
Shawn Pearce
ce290b9dc1 Merge "Add extension point for receive-pack initialization." 2014-01-31 22:32:39 +00:00
David Ostrovsky
c2e42230fd Fix build
063e0fd4fd688c1963e5e0402bee1d27b57db009 removed
gerrit-acceptance-tests/configs rule, but missed to update
tools/eclipse/BUCK file.

Change-Id: Ia270966a9d81cc69257dfa1b15efc089d8d6121b
2014-01-31 18:27:09 +01:00
Saša Živkov
ec85a070b3 Add extension point for receive-pack initialization.
Plugins can register ReceivePackInitializer's in order to provide custom
initialization of the ReceivePack instance. For example, the quota plugin
may use this extension point to set the maximum allowed pack size based
on its quota configuration and the disk space occupied by that project.

Change-Id: I8d6e7d4bb75099b0fa5f6968ae5371883bf865ee
2014-01-31 16:38:13 +01:00
Edwin Kempin
792c8d80f5 Allow plugins to display errors in the Gerrit ErrorDialog
Add a method to the JavaScript API to open the Gerrit ErrorDialog with
a given message. Plugins don't want to reimplement the ErrorDialog.

Change-Id: Iec18fb873bfbe9f47684414aacbaf697076d315f
Signed-off-by: Edwin Kempin <edwin.kempin@sap.com>
2014-01-31 16:16:52 +01:00
Dave Borowitz
6ac5b4eef5 Merge changes If0e54f9e,I0146eeef,I307cc64d,I94a71fb1
* changes:
  notedb: Use Patch-set instead of Patch-Set
  SubmitRecord: Add equals and hashCode methods
  Support multiple ChangeUpdate operations in a batch
  Inject MetaDataUpdate/ChangeUpdate in ChangeNotesTest
2014-01-31 14:50:26 +00:00
Dave Borowitz
6237151502 Merge "notedb: Use user/account pair as author instead of committer" 2014-01-31 14:48:53 +00:00
David Pursehouse
3bac1066d0 Merge "ReviewerPredicate: Filter out draft changes when DRAFT workflow is disabled" 2014-01-31 13:20:01 +00:00
Edwin Kempin
e2f0bccf4b Merge "Support AcceptsPost for RestCollections in plugins" 2014-01-31 07:54:36 +00:00
David Ostrovsky
3f5dc3faeb Add ChangeApi.get() method
Change-Id: I9862a2475781a37c6b964d2a977b3d187d44cf09
2014-01-31 08:16:31 +01:00
David Ostrovsky
4030c6f075 Consistently apply camel case naming convention in server code
Change-Id: I213e4a58f58799d095baadd33921755e5b07f8f6
2014-01-31 07:02:14 +00:00
Dave Borowitz
063e0fd4fd Run acceptance tests with different configs using @Parameterized
This is a simpler approach as it doesn't require generating extra test
targets for Buck, nor rebuilding test jars to get everything working
in Eclipse. There are enough tests running at any given time with Buck
that having each acceptance test take twice as long should maintain
adequate parallelism.

Change-Id: I109acaf6b957e8f9dd621e98abe46a75f78cdc32
2014-01-30 16:11:36 -08:00
Dave Borowitz
a958192606 notedb: Use Patch-set instead of Patch-Set
Other standard git footers (e.g. Signed-off-by) only capitalize the
first word.

Change-Id: If0e54f9eb088a706bf6a919971505584eb9df330
2014-01-30 15:14:41 -08:00
Dave Borowitz
17c9a59e81 SubmitRecord: Add equals and hashCode methods
Change-Id: I0146eeefa4b552e5e39d6138fc6285a3a017a046
2014-01-30 15:14:41 -08:00
Dave Borowitz
39d7b44967 notedb: Use user/account pair as author instead of committer
It will be possible in the future for a single end-user action to
cause the server to make modifications to a change on behalf of
someone not the immediate end user, e.g. normalizing labels on submit.
To represent this, we would have multiple commits in a single update
with different authors. For consistency, use the server identity for
all resultant committers.

Change-Id: I6930d1da3125b724af53514549a9408086bbb7fd
2014-01-30 15:14:30 -08:00
Dave Borowitz
1b93f8bd68 Support multiple ChangeUpdate operations in a batch
Change-Id: I307cc64d38d609fb9e1613dee01f87f5b953dbe7
2014-01-30 15:14:30 -08:00
Dave Borowitz
3d476bc5a0 Inject MetaDataUpdate/ChangeUpdate in ChangeNotesTest
Change-Id: I94a71fb1f1b4a289d4f0eba9d6471918c856a67c
2014-01-30 15:14:30 -08:00
David Ostrovsky
ca2c7f5325 Plugin API: Add ChangeStatus
Change-Id: I5962451476b6de1fec7915026120294b9a1c8f32
2014-01-30 21:40:55 +01:00
Dave Borowitz
02bf1cd82a Upgrade guava to 16.0
Change-Id: I810574fe5f6ed5c397b595092105d970eebfc3a2
2014-01-30 20:28:00 +00:00
David Pursehouse
f81d3d127b Update replication plugin to latest revision
To get the following fixes and updates:

- Change default push refSpec to 'refs/*:refs/*' (non-forced push)
- Allow default force update to be configurable
- Correct misspelled method name

Change-Id: Ic734a7290a303562e42b11a0b64697a0fcbc1d71
2014-01-30 10:37:03 +09:00
Shawn Pearce
2aa28ce780 Merge "ChangeScreen2: list download commands in migration doc" 2014-01-30 00:50:09 +00:00
Jonathan Nieder
5ae652b54b ChangeScreen2: list download commands in migration doc
Explicitly mention checkout, pull, and cherry-pick in the intro to the
new change screen to make it easier for people used to the old change
screen to find their new location.

Change-Id: I1cc687624a6b6486ae899e16612727e07b338ee2
2014-01-29 16:43:31 -08:00
Shawn Pearce
f066bcce28 Merge "New replication config default in 2.9 Release Notes" 2014-01-30 00:38:44 +00:00
Shawn Pearce
ca86d74331 Merge "ChangeScreen2: Show on which categories a reviewer can vote" 2014-01-30 00:37:38 +00:00
Luca Milanesio
eb64932602 New replication config default in 2.9 Release Notes
Change-Id: I3014296dcf0cd965c4605fd933dfcb516573862a
See: https://gerrit-review.googlesource.com/52363
2014-01-30 00:30:09 +00:00
David Ostrovsky
779ea22381 ChangeScreen2: Show on which categories a reviewer can vote
Old change screen had grey styled cells in approval table for labels a
reviewer hadn't right to vote on. The new change screen uses chips to
present the reviewers name and therefore this valuable information is lost.

Add it back in form of tooltip with the list of votable labels for each
reviewer.

Bug: issue 2078
Change-Id: I0f101e43c625bfe1f181b46574c6677d930a5c19
2014-01-30 00:58:46 +01:00
Edwin Kempin
0d99c57e8e Merge "Change GWT plugin archetype to implement screen instead of dialog" 2014-01-29 21:38:10 +00:00
Edwin Kempin
42d89529d8 Support AcceptsPost for RestCollections in plugins
If a RestCollection in a plugin implements AcceptsPost POST requests
for it currently fail with Not Found.

Change-Id: I6db50cbe08c5ec53dac727d90af58fbe4538dbcf
Signed-off-by: Edwin Kempin <edwin.kempin@sap.com>
2014-01-29 21:25:24 +01:00
Shawn Pearce
c792114e50 Merge "Plugin API: Add factory methods for frequently used review operations" 2014-01-29 20:18:58 +00:00
David Ostrovsky
58aa6d58ab ReviewerPredicate: Filter out draft changes when DRAFT workflow is disabled
Change-Id: Icd11184d893d0a32c2112534d05f053acdf8a700
2014-01-29 18:04:15 +01:00
Eryk Szymanski
945fbd4bea Removed unnecessary Math.round() call
Result of the operation is always an integer so the call to Math.round() 
is redundant.

Change-Id: I19fe01b0d71241012ad9f2b5f5c11fabf2b216f5
Signed-off-by: Eryk Szymanski <eryksz@gmail.com>
2014-01-28 23:25:09 +00:00
Shawn Pearce
5579c284d5 Make CSS class names stable
GWT's class obfuscation makes it impossible for users to write their
own CSS that won't break on upgrade. This further helps preserve
class names so users can continue to rely on the names and use
GerritSite.css to its full ability.

Change-Id: I10acf06c150daebf9d74ba8ad101c9643392781a
2014-01-28 12:56:53 -08:00
Shawn Pearce
97ba4a9b3b Merge branch 'stable-2.8'
* stable-2.8:
  SideBySide2: Fix truncated long lines

Conflicts:
	gerrit-gwtui/src/main/java/com/google/gerrit/client/diff/SideBySide2.java

Change-Id: I36477d17261745150ebc4799676bb953584f74e5
2014-01-28 11:53:04 -08:00
Edwin Kempin
66091fc02d Change GWT plugin archetype to implement screen instead of dialog
We are now able to display a GWT screen when clicking on a menu item.
The GWT plugin archetype should create an example screen. So far it
was only showing a dialog.

Change-Id: I8b599542ae92e01195ce0402ae085615e86a1b62
Signed-off-by: Edwin Kempin <edwin.kempin@sap.com>
2014-01-28 20:51:09 +01:00
Shawn Pearce
138bc1443a Merge "Allow plugins to check for config updates when reading plugin config" 2014-01-28 19:50:14 +00:00
Edwin Kempin
9ed9540325 Allow plugins to check for config updates when reading plugin config
Plugins can have their global configuration in a plugin subsection of
the gerrit.config file. PluginConfigFactory provides easy access to
this global plugin configuration and ensures that the gerrit.config
file is not on any access loaded from the filesystem. The
GerritServerConfig is injected into the PluginConfigFactory
so that updates of the gerrit.config file only get visible in
PluginConfigFactory after the server has been restarted.

Plugins may want to provide ways to live update their global
configuration, e.g. they could provide a REST endpoint to update
their plugin section in gerrit.config. To make the changes visible
in PluginConfigFactory the cached GerritServerConfig in it needs
to be reloaded. For this purpose this change adds a refresh flag
to the method in PluginConfigFactory that reads the global plugin
config. The GerritServerConfig is then reloaded when it is was
changed in the filesystem.

Change-Id: Ide6b1457bfe5a63459e816ca9e00604a62da8365
Signed-off-by: Edwin Kempin <edwin.kempin@sap.com>
2014-01-28 11:49:47 -08:00
Shawn Pearce
4cf09ed6d4 Merge "Support warning on project plugin configuration parameter" 2014-01-28 19:44:18 +00:00
David Ostrovsky
1b61dc74d0 Add configuration to disable DRAFT workflow
Draft workflow breaks replication and some sites might want to disable it
for all projects. While it is already possible to block new draft changes
from be uploaded, it is still possible to delete existing drafts.

This change disable draft workflow completely.

Change-Id: Iabae835a26caab5ea4c10701c75c9b0b84882f86
2014-01-28 19:13:34 +01:00
Edwin Kempin
6601f2db97 Merge "Append ':' to label of project config param if info icon is displayed" 2014-01-28 16:54:48 +00:00
Edwin Kempin
9e64561dba Update year to 2014 in license headers of plugin archetype resources
The Maven plugin archetypes are used to create new plugin projects.
The files generated for these new plugin projects should have the
correct year in the license headers.

Change-Id: I86919f19191089e83ba9111d86a29d2798b75feb
Signed-off-by: Edwin Kempin <edwin.kempin@sap.com>
2014-01-28 16:09:15 +01:00
Edwin Kempin
aec613230a Support warning on project plugin configuration parameter
For project plugin configuration parameter it is now possible to
provide a warning. If set this warning is shown as a tooltip on a
warning icon that is displayed for the project plugin configuration
parameter on the ProjectInfoScreen.

E.g. a warning could be displayed when a configuration parameter is
not editable to explain why this parameter cannot be edited. For
example the Jira plugin allows to enabled/disabled the Jira
integration for a project from the ProjectInfoScreen. This parameter
is not editable if the Jira integration is enforced by a parent
project. In this case it makes sense to display a message which says
that the Jira integration cannot be disabled for this project because
the Jira integration is enforced by parent project X.

The warning icon is taken from the Tango Icon Library [1].

[1] http://tango.freedesktop.org/Tango_Icon_Library

Change-Id: I15d1bec7eaceced487c22202b2514b09dc7c50f8
Signed-off-by: Edwin Kempin <edwin.kempin@sap.com>
2014-01-28 13:20:33 +01:00
Edwin Kempin
7cdc7f149f Append ':' to label of project config param if info icon is displayed
Change-Id: I1e92adb4d115cd33817d814805e9c6e59437edea
Signed-off-by: Edwin Kempin <edwin.kempin@sap.com>
2014-01-28 13:20:19 +01:00
Shawn Pearce
6c99b232da SideBySide2: Fix truncated long lines
CM3 needs a call to refresh() after setHeight to get the horizontal
width correct for any lines longer than the window width.

Bug: Issue 2428
Change-Id: I608264d71c98829c2f39d9acef12445088591bbb
2014-01-28 10:01:09 +00:00
David Pursehouse
0fb9ebcd7b Merge branch 'stable-2.8'
* stable-2.8:
  SideBySide2: Fix copy in Firefox and Safari

Conflicts:
	gerrit-gwtui/src/main/java/com/google/gerrit/client/diff/SideBySide2.java

Change-Id: I6c7520a987c64e9de2c7146cc87d751e3db48bb3
2014-01-28 18:07:25 +09:00
Shawn Pearce
527200c83d SideBySide2: Fix copy in Firefox and Safari
Ctrl-C/Cmd-C should be copy to clipboard and should not activate the
insert new comment feature of Gerrit.  Bind to the C key not 'c'.
This uses a different binding method inside of CM3 allowing the
browser to still handle Ctrl-C/Cmd-C.

Bug: issue 2416
Change-Id: Iad6d4e8502c1766caf1cc897f1398068bbde7155
2014-01-28 17:37:50 +09:00
David Pursehouse
8905dd4b96 Remove unused imports in acceptance tests
Change-Id: I3af378b572ad84332fda5bab8c7770b3144a1118
2014-01-28 17:30:44 +09:00
David Pursehouse
e994c16df9 Merge branch 'stable-2.8'
* stable-2.8:
  Only add "cherry picked from" when cherry picking a merged change

Change-Id: Ia07a4dc78e96eeb2b439329c92c73be8f73c2e98
2014-01-28 17:11:59 +09:00