Ignore:
Timestamp:
2016-04-09T16:08:55+02:00 (9 years ago)
Author:
Don-vip
Message:

refactor and move changeset download tasks to actions.downloadtasks package

Location:
trunk/src/org/openstreetmap/josm/gui/dialogs
Files:
7 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/gui/dialogs/ChangesetDialog.java

    r10055 r10124  
    3535import org.openstreetmap.josm.Main;
    3636import org.openstreetmap.josm.actions.AbstractInfoAction;
     37import org.openstreetmap.josm.actions.downloadtasks.ChangesetHeaderDownloadTask;
     38import org.openstreetmap.josm.actions.downloadtasks.PostDownloadHandler;
    3739import org.openstreetmap.josm.data.osm.Changeset;
    3840import org.openstreetmap.josm.data.osm.ChangesetCache;
    … …  
    4446import org.openstreetmap.josm.gui.SideButton;
    4547import org.openstreetmap.josm.gui.dialogs.changeset.ChangesetCacheManager;
    46 import org.openstreetmap.josm.gui.dialogs.changeset.ChangesetHeaderDownloadTask;
    4748import org.openstreetmap.josm.gui.dialogs.changeset.ChangesetInSelectionListModel;
    4849import org.openstreetmap.josm.gui.dialogs.changeset.ChangesetListCellRenderer;
    … …  
    330331
    331332        @Override
    332         public void itemStateChanged(ItemEvent arg0) {
     333        public void itemStateChanged(ItemEvent e) {
    333334            updateEnabledState();
    334335
    … …  
    354355
    355356        @Override
    356         public void actionPerformed(ActionEvent arg0) {
     357        public void actionPerformed(ActionEvent e) {
    357358            ChangesetListModel model = getCurrentChangesetListModel();
    358359            Set<Integer> sel = model.getSelectedChangesetIds();
    … …  
    360361                return;
    361362            ChangesetHeaderDownloadTask task = new ChangesetHeaderDownloadTask(sel);
    362             Main.worker.submit(task);
     363            Main.worker.submit(new PostDownloadHandler(task, task.download()));
    363364        }
    364365
    … …  
    368369
    369370        @Override
    370         public void itemStateChanged(ItemEvent arg0) {
    371             updateEnabledState();
    372 
     371        public void itemStateChanged(ItemEvent e) {
     372            updateEnabledState();
    373373        }
    374374
    … …  
    392392
    393393        @Override
    394         public void actionPerformed(ActionEvent arg0) {
     394        public void actionPerformed(ActionEvent e) {
    395395            List<Changeset> sel = getCurrentChangesetListModel().getSelectedOpenChangesets();
    396396            if (sel.isEmpty())
    … …  
    404404
    405405        @Override
    406         public void itemStateChanged(ItemEvent arg0) {
     406        public void itemStateChanged(ItemEvent e) {
    407407            updateEnabledState();
    408408        }
    … …  
    427427
    428428        @Override
    429         public void actionPerformed(ActionEvent arg0) {
     429        public void actionPerformed(ActionEvent e) {
    430430            Set<Changeset> sel = getCurrentChangesetListModel().getSelectedChangesets();
    431431            if (sel.isEmpty())
    … …  
    435435            String baseUrl = Main.getBaseBrowseUrl();
    436436            for (Changeset cs: sel) {
    437                 String url = baseUrl + "/changeset/" + cs.getId();
    438                 OpenBrowser.displayUrl(
    439                         url
    440                 );
     437                OpenBrowser.displayUrl(baseUrl + "/changeset/" + cs.getId());
    441438            }
    442439        }
    … …  
    447444
    448445        @Override
    449         public void itemStateChanged(ItemEvent arg0) {
     446        public void itemStateChanged(ItemEvent e) {
    450447            updateEnabledState();
    451448        }
    … …  
    469466
    470467        @Override
    471         public void actionPerformed(ActionEvent arg0) {
     468        public void actionPerformed(ActionEvent e) {
    472469            ChangesetListModel model = getCurrentChangesetListModel();
    473470            Set<Integer> sel = model.getSelectedChangesetIds();
    … …  
    521518            } else {
    522519                task = new ChangesetHeaderDownloadTask(toDownload);
    523                 future = Main.worker.submit(task);
     520                future = Main.worker.submit(new PostDownloadHandler(task, task.download()));
    524521            }
    525522

  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/ChangesetCacheManager.java

    r9916 r10124  
    4040
    4141import org.openstreetmap.josm.Main;
     42import org.openstreetmap.josm.actions.downloadtasks.AbstractChangesetDownloadTask;
     43import org.openstreetmap.josm.actions.downloadtasks.ChangesetContentDownloadTask;
     44import org.openstreetmap.josm.actions.downloadtasks.ChangesetHeaderDownloadTask;
     45import org.openstreetmap.josm.actions.downloadtasks.ChangesetQueryTask;
     46import org.openstreetmap.josm.actions.downloadtasks.PostDownloadHandler;
    4247import org.openstreetmap.josm.data.osm.Changeset;
    4348import org.openstreetmap.josm.data.osm.ChangesetCache;
    … …  
    4651import org.openstreetmap.josm.gui.SideButton;
    4752import org.openstreetmap.josm.gui.dialogs.changeset.query.ChangesetQueryDialog;
    48 import org.openstreetmap.josm.gui.dialogs.changeset.query.ChangesetQueryTask;
    4953import org.openstreetmap.josm.gui.help.ContextSensitiveHelpAction;
    5054import org.openstreetmap.josm.gui.help.HelpUtil;
    … …  
    652656    public void setSelectedChangesets(Collection<Changeset> changesets) {
    653657        model.setSelectedChangesets(changesets);
    654         int idx = model.getSelectionModel().getMinSelectionIndex();
    655         if (idx < 0) return;
    656         tblChangesets.scrollRectToVisible(tblChangesets.getCellRect(idx, 0, true));
     658        final int idx = model.getSelectionModel().getMinSelectionIndex();
     659        if (idx < 0)
     660            return;
     661        GuiHelper.runInEDTAndWait(new Runnable() {
     662            @Override
     663            public void run() {
     664                tblChangesets.scrollRectToVisible(tblChangesets.getCellRect(idx, 0, true));
     665            }
     666        });
    657667        repaint();
    658668    }
    … …  
    696706     * @param task The changeset download task to run
    697707     */
    698     public void runDownloadTask(final ChangesetDownloadTask task) {
    699         Main.worker.submit(task);
     708    public void runDownloadTask(final AbstractChangesetDownloadTask task) {
     709        Main.worker.submit(new PostDownloadHandler(task, task.download()));
    700710        Main.worker.submit(new Runnable() {
    701711            @Override
    702712            public void run() {
    703                 if (task.isCanceled() || task.isFailed()) return;
    704                 setSelectedChangesets(task.getDownloadedChangesets());
     713                if (task.isCanceled() || task.isFailed())
     714                    return;
     715                setSelectedChangesets(task.getDownloadedData());
    705716            }
    706717        });

  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/ChangesetContentPanel.java

    r10055 r10124  
    3434import org.openstreetmap.josm.Main;
    3535import org.openstreetmap.josm.actions.AutoScaleAction;
     36import org.openstreetmap.josm.actions.downloadtasks.ChangesetContentDownloadTask;
    3637import org.openstreetmap.josm.data.osm.Changeset;
    3738import org.openstreetmap.josm.data.osm.OsmPrimitive;

  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/ChangesetDetailPanel.java

    r9493 r10124  
    2929import org.openstreetmap.josm.Main;
    3030import org.openstreetmap.josm.actions.AutoScaleAction;
     31import org.openstreetmap.josm.actions.downloadtasks.ChangesetHeaderDownloadTask;
     32import org.openstreetmap.josm.actions.downloadtasks.PostDownloadHandler;
    3133import org.openstreetmap.josm.data.osm.Changeset;
    3234import org.openstreetmap.josm.data.osm.ChangesetCache;
    … …  
    298300        @Override
    299301        public void actionPerformed(ActionEvent evt) {
    300             if (currentChangeset == null) return;
    301             Main.worker.submit(
    302                     new ChangesetHeaderDownloadTask(
    303                             ChangesetDetailPanel.this,
    304                             Collections.singleton(currentChangeset.getId())
    305                     )
     302            if (currentChangeset == null)
     303                return;
     304            ChangesetHeaderDownloadTask task = new ChangesetHeaderDownloadTask(
     305                    ChangesetDetailPanel.this,
     306                    Collections.singleton(currentChangeset.getId())
    306307            );
     308            Main.worker.submit(new PostDownloadHandler(task, task.download()));
    307309        }
    308310
    … …  
    313315
    314316    /**
    315      * Selects the primitives in the content of this changeset in the current
    316      * data layer.
     317     * Selects the primitives in the content of this changeset in the current data layer.
    317318     *
    318319     */

  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/ChangesetDiscussionPanel.java

    r8840 r10124  
    2121
    2222import org.openstreetmap.josm.Main;
     23import org.openstreetmap.josm.actions.downloadtasks.ChangesetHeaderDownloadTask;
     24import org.openstreetmap.josm.actions.downloadtasks.PostDownloadHandler;
    2325import org.openstreetmap.josm.data.osm.Changeset;
    2426import org.openstreetmap.josm.io.OnlineResource;
    … …  
    6870        @Override
    6971        public void actionPerformed(ActionEvent evt) {
    70             if (current == null) return;
    71             Main.worker.submit(
    72                     new ChangesetHeaderDownloadTask(
    73                             ChangesetDiscussionPanel.this,
    74                             Collections.singleton(current.getId()),
    75                             true /* include discussion */
    76                     )
     72            if (current == null)
     73                return;
     74            ChangesetHeaderDownloadTask task = new ChangesetHeaderDownloadTask(
     75                    ChangesetDiscussionPanel.this,
     76                    Collections.singleton(current.getId()),
     77                    true /* include discussion */
    7778            );
     79            Main.worker.submit(new PostDownloadHandler(task, task.download()));
    7880        }
    7981

  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/DownloadChangesetContentAction.java

    r9665 r10124  
    99import javax.swing.AbstractAction;
    1010
     11import org.openstreetmap.josm.actions.downloadtasks.ChangesetContentDownloadTask;
    1112import org.openstreetmap.josm.tools.CheckParameterUtil;
    1213

  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/SingleChangesetDownloadPanel.java

    r9059 r10124  
    1717
    1818import org.openstreetmap.josm.Main;
     19import org.openstreetmap.josm.actions.downloadtasks.ChangesetContentDownloadTask;
    1920import org.openstreetmap.josm.gui.SideButton;
    2021import org.openstreetmap.josm.gui.widgets.ChangesetIdTextField;
Note: See TracChangeset for help on using the changeset viewer.