Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support multiple amendments in a single line #6

Open
badicsalex opened this issue Nov 5, 2022 · 0 comments
Open

Support multiple amendments in a single line #6

badicsalex opened this issue Nov 5, 2022 · 0 comments

Comments

@badicsalex
Copy link
Owner

badicsalex commented Nov 5, 2022

Multiple different positions and texts:

A megújuló energia közlekedési célú felhasználásának előmozdításáról és a közlekedésben felhasznált energia üvegházhatású gázkibocsátásának csökkentéséről szóló 2010. évi CXVII. törvény 13. § (2) bekezdés a) pontjában a „miniszterrel együttesen” szövegrész helyébe a „miniszter egyetértésével” szöveg, 13. § (2) bekezdés b) pontjában a „miniszterrel” szövegrész helyébe a „miniszter egyetértésével” szöveg lép.
(2010. évi CXXX. törvény 44. § (13))

Even different types:

A jogalkotásról szóló 2010. évi CXXX. törvény 11. alcímének címében és 25. § (1) bekezdésében az „A Magyar Köztársaság” szövegrész helyébe a „Magyarország” szöveg lép.
(2011. évi CCI. törvény 377. § e) pont)

Multiple repeals:

Hatályát veszti a légi-, a vasúti és a víziközlekedési balesetek és egyéb közlekedési események szakmai vizsgálatáról szóló 2005. évi CLXXXIV. törvény 22. § (2) bekezdésében az „együttes rendeletben állapítsák meg” szövegrész, 22. § (3) bekezdésében az „– a miniszterrel egyetértésben – együttes rendeletben állapítsák meg” szövegrész.

The most probable solution is that SpecialPhrase will need to be refactored, so that it can contain multiple text amendments and repeals, even mixed. At that point, TextAmendment and friends should be simplified so they don't contain any vectors, and are fully expanded position/replacement pairs.

An alternative solution is to keep TextAmendment and Repeal, but these should be vectors of simplified entries. This makes sense from a parsing standpoint, because these are pretty much never mixed, but this distinction is not important down the line in processing.

badicsalex added a commit that referenced this issue Nov 11, 2022
This seems wasteful, but:
- It's way easier to process for an amendment applier
- The files are gzipped anyways

This is a big step towards #6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant