• 0 Posts
  • 34 Comments
Joined 4 years ago
cake
Cake day: June 24th, 2020

help-circle








  • Related stories:

    Fewer international students to study in Canada over next two years

    Fewer international students in the fall could impact Maritime work force

    Maritime universities, students, governments share concern after Ottawa unveils plan to cap student visas

    Also I can’t seem to find a link to the actual survey, but:

    The survey also shows 61 per cent agree so many international students are being admitted into Canada due to mismanaged finances by post-secondary institutions in the country.

    This sounds like a bullshit survey question. This article is all survey and zero data.








  • It’s not just the visible complexity in this one file. The point of it is to keep a subscriber count in sync, but you have that code I referenced above, plus:

    LinkPersonCommunityCreatedEvent LinkPersonCommunityDeletedEvent LinkPersonCommunityCreatedPublisher LinkPersonCommunityDeletedPublisher

    And then there are things like LinkPersonCommunityUpdated[Event/Publisher] which don’t even seem to be used.

    This is all boilerplate IMO.

    And all of that only (currently) serves keeping that subscriber count up to date.

    And then there’s the hidden complexity of how things get wired up with spring.

    And after all that it’s still fragile because that event is not tied to object creation:

      @Transactional
      public void addLink(Person person, Community community, LinkPersonCommunityType type) {
    
        final LinkPersonCommunity newLink = LinkPersonCommunity.builder().community(community)
            .person(person).linkType(type).build();
        person.getLinkPersonCommunity().add(newLink);
        community.getLinkPersonCommunity().add(newLink);
        linkPersonCommunityRepository.save(newLink);
        linkPersonCommunityCreatedPublisher.publish(newLink);
      }
    

    And there’s some code here:

    https://github.com/sublinks/sublinks-api/blob/main/src/main/java/com/sublinks/sublinksapi/api/lemmy/v3/community/controllers/CommunityOwnerController.java#L138C31-L138C50

        final Set linkPersonCommunities = new LinkedHashSet<>();
        linkPersonCommunities.add(LinkPersonCommunity.builder().community(community).person(person)
            .linkType(LinkPersonCommunityType.owner).build());
        linkPersonCommunities.add(LinkPersonCommunity.builder().community(community).person(person)
            .linkType(LinkPersonCommunityType.follower).build());
    
        communityService.createCommunity(community);
    
        linkPersonCommunityRepository.saveAllAndFlush(linkPersonCommunities);
    

    that is able to bypass the community link service and create links in the repository directly, which would presumably not trigger than event.

    Maybe there’s a good reason for that, but it sure looks fragile to me.



  • Browsing the code makes me angry at how bloated Java projects are:

    package com.sublinks.sublinksapi.community.repositories;
    
    import com.sublinks.sublinksapi.community.dto.Community;
    import com.sublinks.sublinksapi.community.models.CommunitySearchCriteria;
    import com.sublinks.sublinksapi.post.dto.Post;
    import com.sublinks.sublinksapi.post.models.PostSearchCriteria;
    import org.springframework.data.domain.Page;
    import org.springframework.data.domain.Pageable;
    import org.springframework.data.jpa.repository.JpaRepository;
    import org.springframework.data.jpa.repository.Query;
    import org.springframework.data.repository.query.Param;
    import java.util.List;
    
    public interface CommunitySearchRepository {
    
      List allCommunitiesBySearchCriteria(CommunitySearchCriteria communitySearchCriteria);
    
    }
    

    Every file is 8 directories deep, has 20 imports, and one SQL statement embedded in a string literal. 😭