Closes #3433697: Drupal 11 compatibility Fixes
Closes #3433697
Merge request reports
Activity
1 1 name: OpenAPI 2 2 type: module 3 3 description: Connects Drupal to the OpenAPI Javascript Library 4 core: 8.x 5 core_version_requirement: ^8 || ^9 || ^10 4 core_version_requirement: ^10.1 || ^11 The End of Life (EOL) for both Drupal 8 and Drupal 9, and as a result, the project update bot has discontinued its support for these versions.
If it is necessary to maintain support for these versions, we can certainly consider reinstating them. I will leave the ultimate decision up to the discretion of the maintainer.
changed this line in version 5 of the diff
19 19 */ 20 20 public $openapiGeneratorManager; 21 21 22 /** 23 * Current Generator plugin manager. 24 * 25 * @var \Drupal\Component\Plugin\PluginManagerInterface 26 */ 27 public $openapiUiManager; changed this line in version 13 of the diff
45 43 '#type' => 'openapi_ui', 46 44 '#openapi_ui_plugin' => $openapi_ui, 47 45 '#openapi_schema' => $openapi_generator->getSpecification(), 48 // '#openapi_schema' => Url::fromRoute('openapi.download', ['openapi_generator' => $openapi_generator->getPluginId()], ['query' => ['_format' => 'json', 'options' => $options]]), 46 '#openapi_schema' => Url::fromRoute('openapi.download', [ 47 'openapi_generator' => $openapi_generator->getPluginId(), 48 ], 49 [ 50 'query' => [ 51 '_format' => 'json', 52 'options' => $options, 53 ], 54 ] 55 ), The line of code was mistakenly uncommented while addressing PHPCS warnings. It has now been commented out again. I can see this code is replaced with the OpenApiGeneratorInterface's getSpecification() method on above line.
Edited by Pravin Gaikwadchanged this line in version 15 of the diff
28 28 * 29 29 * @var string 30 30 */ 31 static $DEFINITION_SEPARATOR = ':'; 31 protected static $definitionSeparator = ':'; changed this line in version 13 of the diff
302 304 return $security_definitions; 303 305 } 304 306 307 /** 308 * Helper to set security. 309 */ 305 310 public function getSecurity() { 22 22 * Set to TRUE to run this test to generate expectation files. 23 23 * 24 24 * The test will be marked as a fail when generating test files. 25 * 26 * @var bool 25 27 */ 26 28 protected static $generateExpectationFiles = FALSE; changed this line in version 13 of the diff
131 136 } 132 137 133 138 /** 134 * Dataprovider for testRequests. 139 * Data provider for testRequests. 135 140 */ 136 public function providerRequestTypes() { 141 public static function providerRequestTypes() { changed this line in version 13 of the diff
241 246 $supported_security_types = ['basic', 'apiKey', 'cookie', 'oauth', 'oauth2']; 242 247 foreach ($security_definitions as $definition_id => $definition) { 243 248 if ($definition_id !== 'csrf_token') { 244 // CSRF Token will never have an auth collector, all others shoud. 249 // CSRF Token will never have an auth collector, all others should. 245 250 $this->assertTrue(array_key_exists($definition_id, $auth_providers), 'Security definition ' . $definition_id . ' not an auth collector.'); 246 251 } 247 $this->assertTrue(in_array($definition['type'], $supported_security_types), 'Security definition schema ' . $definition_id . ' has invalid type '. $definition['type']); 252 $this->assertTrue(in_array($definition['type'], $supported_security_types), 'Security definition schema ' . $definition_id . ' has invalid type ' . $definition['type']); changed this line in version 13 of the diff
- .cspell.json 0 → 100644
1 { 2 "version": "0.2", 3 "language": "en", 4 "ignoreWords": [ 5 "extention", changed this line in version 13 of the diff
- .gitlab-ci.yml 0 → 100644
1 include: This file doesn't look like the GitLab CI template provided by the DA...? Can we base it onhttps://git.drupalcode.org/project/gitlab_templates/-/blob/1.0.x/gitlab-ci/template.gitlab-ci.yml?ref_type=heads, including all the extra explanatory comments? Just for consistency?
55 55 This can be downloaded and configured to display docs within a drupal site using 56 56 the [Redoc for OpenAPI UI](https://drupal.org/project/openapi_ui_redoc) module. 57 57 Once the module installed, you will need to have a supported api schema module 58 nstalled, see "Using OpenAPI" above. You can then navigate to the respective url 59 for the api. 58 installed, see "Using OpenAPI" above. You can then navigate to the respective 59 url for the api. changed this line in version 13 of the diff
added 1 commit
- 7a8e0332 - Reverted change from fix phpcs warning commit.
@phenaproxima could you please review this MR and merge if possible!!
- Resolved by Adam G-H
- Resolved by Adam G-H
- Resolved by Adam G-H
- Resolved by Adam G-H
- Resolved by Vishal Khode
- Resolved by Adam G-H
added 1 commit
- 21101538 - Remove testing for previous major, since I decided to drop support