Skip to content
Snippets Groups Projects

ISAICP-9062: Fix the state of OSS Catalogue

Merged Claudiu Cristea requested to merge digit/digit-joinup-dev:ISAICP-9062 into master
3 unresolved threads

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
137 152 */
138 153 public function disableEuOssCatalogueSubmitForm(): void {
139 154 $this->state->set('joinup.oss_catalogue_is_disabled', TRUE);
155 $uri = OssCatalogueCollectionInterface::COLLECTION_ENTITY_ID;
156 $collection = $this->entityTypeManager->getStorage('rdf_entity')->load($uri);
157 $collection->set('field_ar_state', 'draft')->save();
  • 137 152 */
    138 153 public function disableEuOssCatalogueSubmitForm(): void {
    139 154 $this->state->set('joinup.oss_catalogue_is_disabled', TRUE);
    155 $uri = OssCatalogueCollectionInterface::COLLECTION_ENTITY_ID;
    156 $collection = $this->entityTypeManager->getStorage('rdf_entity')->load($uri);
  • 7 7 | title | oss_development_status | oss_source | oss_categories | oss_short_description | status |
    8 8 | The Opensource Panacea | development | developers_italia | content-management | The cure | published |
    9 9
    10 When I go to "/eu-oss-catalogue"
  • added 1 commit

    • facd13d2 - ISAICP-9062: Enable functionality during tests.

    Compare with previous version

  • added 1 commit

    • 840a2561 - ISAICP-9062: Enable functionality during tests.

    Compare with previous version

  • added 1 commit

    • 921d68d5 - ISAICP-9062: Enable functionality during tests.

    Compare with previous version

  • added 1 commit

    • f6231a16 - ISAICP-9062: Rely on collection state.

    Compare with previous version

  • I have changed the philosophy. We should not have 2 switches: one the state var, the other the state of the collection. If we need to switch the collection state let's rely on that and drop the state var

  • Hervé Donner approved this merge request

    approved this merge request

  • Please register or sign in to reply
    Loading