Merge pull request #2896 from hotosm/develop

Release candidate 4.0.4

Author: pantierra

backend/api/projects/resources.py

@@ -32,6 +32,7 @@
 
 
 class ProjectsRestAPI(Resource):
+    @token_auth.login_required(optional=True)
     def get(self, project_id):
         """
         Get a specified project including it's area
@@ -91,9 +92,9 @@ def get(self, project_id):
                 request.environ.get("HTTP_ACCEPT_LANGUAGE"),
                 abbreviated,
             )
-            project_dto = project_dto.to_primitive()
 
             if project_dto:
+                project_dto = project_dto.to_primitive()
                 if as_file:
                     return send_file(
                         io.BytesIO(geojson.dumps(project_dto).encode("utf-8")),

backend/services/messaging/smtp_service.py

@@ -37,7 +37,7 @@ def send_contact_admin_email(data):
         if email_to is None:
             raise ValueError("variable TM_EMAIL_CONTACT_ADDRESS not set")
 
-        message = """New contact from {name} <{email}>.
+        message = """New contact from {name} - {email}.
             <p>{content}</p>
             """.format(
             name=data.get("name"), email=data.get("email"), content=data.get("content"),

backend/services/project_search_service.py

@@ -244,13 +244,14 @@ def _filter_projects(search_dto: ProjectSearchDTO):
 
         if search_dto.text_search:
             # We construct an OR search, so any projects that contain or more of the search terms should be returned
-            or_search = search_dto.text_search.replace(" ", " | ")
-
+            or_search = " | ".join(
+                [x for x in search_dto.text_search.split(" ") if x != ""]
+            )
             opts = [
                 ProjectInfo.text_searchable.match(
                     or_search, postgresql_regconfig="english"
                 ),
-                ProjectInfo.name.like(f"%{search_dto.text_search}%"),
+                ProjectInfo.name.like(f"%{or_search}%"),
             ]
             try:
                 opts.append(Project.id == int(search_dto.text_search))
@@ -285,8 +286,7 @@ def _filter_projects(search_dto: ProjectSearchDTO):
             )
             orgs_managed = [org.id for org in user_orgs_list]
             org_projects = query.filter(Project.organisation_id.in_(orgs_managed))
-            my_projects = query.filter(Project.author_id == search_dto.managed_by)
-            query = org_projects.union(team_projects).union(my_projects)
+            query = org_projects.union(team_projects)
 
         all_results = query.all()
         paginated_results = query.paginate(search_dto.page, 14, True)

backend/services/users/authentication_service.py

@@ -18,7 +18,7 @@
 @token_auth.verify_token
 def verify_token(token):
     """ Verify the supplied token and check user role is correct for the requested resource"""
-
+    tm.authenticated_user_id = None
     if not token:
         current_app.logger.debug(f"Token not supplied {request.base_url}")
         return False

frontend/src/assets/styles/_extra.scss

@@ -4,6 +4,9 @@
     max-height: 100vh;
     overflow-y: scroll;
   }
+  .video-popup-content {
+    width: 90% !important;
+  }
   .user-popup-content {
     width: 70% !important;
     left: 15% !important;
@@ -32,6 +35,9 @@
   .popup-content {
     width: 44% !important;
   }
+  .video-popup-content {
+    width: 66% !important;
+  }
   .user-popup-content {
     width: 13% !important;
   }

frontend/src/components/taskSelection/imagery.js

@@ -10,17 +10,17 @@ export function Imagery({ value = '' }: Object) {
   const [isCopied, setCopied] = useCopyClipboard();
 
   const handleClick = () => setCopied(value);
-  let content = <span>{value}</span>;
+  let content = <span title={value}>{value}</span>;
   let copyButton;
   let messageId;
   if (value) {
-    if (value.startsWith('tms[')) {
+    if (value.startsWith('tms')) {
       messageId = 'customTMSLayer';
     }
-    if (value.startsWith('wms[')) {
+    if (value.startsWith('wms')) {
       messageId = 'customWMSLayer';
     }
-    if (value.startsWith('wmts[')) {
+    if (value.startsWith('wmts')) {
       messageId = 'customWMTSLayer';
     }
     if (value.startsWith('http') || value.startsWith('https')) {
@@ -42,7 +42,7 @@ export function Imagery({ value = '' }: Object) {
     content = <FormattedMessage {...messages.noImageryDefined} />;
   }
   return (
-    <p className={`f5 fw6 pt1 ma0 ${value ? 'blue-dark' : 'blue-light'}`}>
+    <p className={`f5 fw6 pt1 pr3 ma0 truncate ${value ? 'blue-dark' : 'blue-light'}`}>
       {content}
       {copyButton}
     </p>

frontend/src/components/taskSelection/tests/imagery.test.js

@@ -16,6 +16,16 @@ it('test if Imagery returns the correct FormattedMessage to TMS', () => {
   expect(testInstance.findByType(FormattedMessage).props.id).toBe('project.imagery.tms');
 });
 
+it('test if Imagery returns the correct FormattedMessage to TMS, even without the zoom level information', () => {
+  const element = createComponentWithIntl(
+    <Imagery
+      value={'tms:https://service.com/earthservice/tms/Layer@EPSG:3857@jpg/{zoom}/{x}/{-y}.jpg'}
+    />,
+  );
+  const testInstance = element.root;
+  expect(testInstance.findByType(FormattedMessage).props.id).toBe('project.imagery.tms');
+});
+
 it('test if Imagery returns the correct FormattedMessage to WMS', () => {
   const element = createComponentWithIntl(
     <Imagery

frontend/src/locales/cs.json

@@ -788,8 +788,8 @@
   "pages.learn.map.title": "Naučte se mapovat",
   "pages.learn.map.intro": "Mapování v OpenStreetMap je týmové úsilí, které neustále roste díky komunitě stovek tisíc globálních přispěvatelů. Každý se může podílet na této otevřené mapě světa a ta zas potřebuje odpovědné lidi, kteří jsou otevření k učení a měli cit k detailům.",
   "pages.learn.map.description": "Nyní můžete začít svou mapovací cestu. Před přidáním nebo úpravou existujících dat nemusíte vyžadovat povolení. Pokud si myslíte, že můžete něco vylepšit, udělejte to. Ale vždy pamatujte, s velkou mocí přichází velká odpovědnost.",
-  "pages.learn.videos.title": "",
-  "pages.learn.manuals.title": "",
+  "pages.learn.videos.title": "Videa",
+  "pages.learn.manuals.title": "Příručky",
   "pages.learn.tutorials.quick_start.title": "Rychlý návod",
   "pages.learn.tutorials.quick_start.description": "Pokyny krok za krokem vám umožní mapovat co nejrychleji.",
   "pages.learn.tutorials.tm_manual.title": "Uživatelská příručka Tasking Manageru",
@@ -804,12 +804,12 @@
   "pages.learn.map.steps.task.description": "Projekty jsou rozděleny do sady úloh, které lze zvládnout co do velikosti a přizpůsobit vaší úrovni dovedností",
   "pages.learn.map.steps.map.title": "Mapujte prostřednictvím OpenStreetMap",
   "pages.learn.map.steps.map.description": "Pokud jste v mapování nováčkem, doporučujeme před zahájením mapování zkontrolovat níže uvedené příručky.",
-  "pages.learn.videos.signup.title": "",
-  "pages.learn.videos.signup.description": "",
-  "pages.learn.videos.map_roads.title": "",
-  "pages.learn.videos.map_roads.description": "",
-  "pages.learn.videos.map_buildings.title": "",
-  "pages.learn.videos.map_buildings.description": "",
+  "pages.learn.videos.signup.title": "Zaregistrujte se do Tasking Manageru",
+  "pages.learn.videos.signup.description": "Naučte se, jak se přihlásit do Tasking Manageru.",
+  "pages.learn.videos.map_roads.title": "Mapování cest",
+  "pages.learn.videos.map_roads.description": "Naučte se, jak mapovat cesty na OpenStreetMap.",
+  "pages.learn.videos.map_buildings.title": "Vyberte úlohu a mapujte budovy",
+  "pages.learn.videos.map_buildings.description": "Naučte se, jak mapovat budovy na OpenStreetMap.",
   "pages.learn.validate_title": "Naučte se validovat",
   "pages.learn.validate.intro": "Validace je důležitou součástí procesu. Vyžaduje důvěru ve vaše mapovací schopnosti a ochotu pomoci koučovat a radit novějším mapovačům.",
   "pages.learn.validate.description": "Získání druhého, třetího nebo čtvrtého páru očí na mapované funkce je důležitým krokem k zajištění kvality dat přidávaných do OpenStreetMap a jejich použití po celém světě.",