Bug 1633149 Comment 2 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

The issue is the logic for finding *relevant* changes in repository, which doesn't work for projects with PC:
https://github.com/mozilla/pontoon/blob/master/pontoon/sync/vcs/models.py#L266



```python
for locale in self.db_project.locales.all():
    for path in changed_files:
        project_files = ProjectFiles(
            locale.code,
            [self.configuration.parsed_configuration],
        )
        m = project_files.match(path)
        if m:
            files.setdefault(m[1], []).append(locale)
```

It also seems bug 1608045 is duplicate of this.
The issue is the logic for finding *relevant* changes in repository, which doesn't work for projects with PC:
https://github.com/mozilla/pontoon/blob/master/pontoon/sync/vcs/models.py#L266



```python
for locale in self.db_project.locales.all():
    project_files = self.configuration.get_or_set_project_files(locale.code)
    for path in changed_files:
        absolute_path = os.path.join(self.source_directory_path, path)
        m = project_files.match(absolute_path)
        if m:
            # Use reference paths
            reference_path = m[1]
            relative_reference_path = reference_path[len(self.source_directory_path) :].lstrip(os.sep)
            files.setdefault(relative_reference_path, []).append(locale)

```

It also seems bug 1608045 is duplicate of this.
The issue is the logic for finding *relevant* changes in repository, which doesn't work for projects with PC:
https://github.com/mozilla/pontoon/blob/master/pontoon/sync/vcs/models.py#L266



```python
for locale in self.db_project.locales.all():
    project_files = self.configuration.get_or_set_project_files(locale.code)
    for path in changed_files:
        absolute_path = os.path.join(self.source_directory_path, path)
        m = project_files.match(absolute_path)
        if m:
            # Use reference paths
            reference_path = m[1]
            relative_reference_path = reference_path[len(self.source_directory_path) :].lstrip(os.sep)
            files.setdefault(relative_reference_path, []).append(locale)

```

It also seems bug 1608045 is a duplicate of this one.

Back to Bug 1633149 Comment 2